-
Notifications
You must be signed in to change notification settings - Fork 35.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Revert "Add patch to make codesign_allocate compatible with Apple's" #20855
Revert "Add patch to make codesign_allocate compatible with Apple's" #20855
Conversation
This reverts commit a4118c6.
ACK a0eb4c5 |
Reference from IRC: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK a0eb4c5
checked-clean-revert ACK a0eb4c5 |
This reverts commit a4118c6. Github-Pull: bitcoin#20855 Rebased-From: a0eb4c5
Backported in #20850 |
…atible with Apple's" a0eb4c5 Revert "Add patch to make codesign_allocate compatible with Apple's" (Pieter Wuille) Pull request description: This reverts bitcoin#20644. It appears that Apple has recently changed their `codesign_allocate` tool back to using 4k alignment on x86_64, at least in some cases, so this patch isn't causing our cctools-based version to be exactly compatible. Furthermore, if codesigning were to change to use https://github.com/achow101/signapple instead, there is no need anymore to try to mimick Apple. ACKs for top commit: laanwj: ACK a0eb4c5 MarcoFalke: checked-clean-revert ACK a0eb4c5 jonasschnelli: ACK a0eb4c5 Tree-SHA512: 529719a76811006122406689233d1e80995107fe1ac1fc862a4ac53ca21685748ed76cac7ca648dd70f0ea43dd8dcf2e29d559beeab10e1d30dc5542ac95fd97
This reverts #20644.
It appears that Apple has recently changed their
codesign_allocate
tool back to using 4k alignment on x86_64, at least in some cases, so this patch isn't causing our cctools-based version to be exactly compatible.Furthermore, if codesigning were to change to use https://github.com/achow101/signapple instead, there is no need anymore to try to mimick Apple.