Skip to content

[capstone] Update Capstone to v4.0.1-49-ge3d5a594#5546

Merged
Rastaban merged 5 commits intomicrosoft:masterfrom
ceeac:cs4
Mar 15, 2019
Merged

[capstone] Update Capstone to v4.0.1-49-ge3d5a594#5546
Rastaban merged 5 commits intomicrosoft:masterfrom
ceeac:cs4

Conversation

@ceeac
Copy link
Contributor

@ceeac ceeac commented Mar 5, 2019

I opted to base the package on commit e3d5a594 (latest v4 branch as of today) rather than v4.0.1 because v4.0.1 does not build successfully with x86_reduce enabled.

This PR also adds support for selectively enabling architecture support and/or diet mode.

Fixes #5276.

@msftclas
Copy link

msftclas commented Mar 5, 2019

CLA assistant check
All CLA requirements met.

@Rastaban
Copy link
Contributor

Rastaban commented Mar 5, 2019

It looks like there are new build breaks on the UWP triplets on windows. Would you be able to look into those?

@Rastaban Rastaban changed the title Update Capstone to v4.0.1-49-ge3d5a594 [capstone] Update Capstone to v4.0.1-49-ge3d5a594 Mar 5, 2019
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you use the full sha here?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

@ceeac
Copy link
Contributor Author

ceeac commented Mar 12, 2019

The failing UWP build is caused by a bug in upstream Capstone; capstone-engine/capstone#1421 should fix this. I'll wait for the upstream PR to be merged and then update straight to the newest v4 branch once the PR is merged.

@Rastaban Rastaban self-assigned this Mar 14, 2019
@Rastaban Rastaban added the wip label Mar 14, 2019
@Rastaban
Copy link
Contributor

Sounds good, I will watch for your update.

@Rastaban Rastaban merged commit 830087d into microsoft:master Mar 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants