fix: local macos bundle signing#2470
Open
bytehow wants to merge 2 commits intoWerWolv:masterfrom
Open
Conversation
- Update entitlements plist path in createPackage macro - Update macos readme to match GitHub actions. Adds missing lib paths, and introduces IMHEX_RESIGN_BUNDLE cmake option
|
Thanks for posting this and updating the build instructions! I was hitting errors trying to do a local build and |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem description
Copying discussion from Discord thread
I'm building on macos, on the head commit on master (
a76eae2c11d2249a7ab6c938cab0dcd8fc750b22) and there seems to be a codesigning issue.System log:
libfreetype is fine from homebrew, but broken in bundle:
Code siging isn't happening at all actually on macOS actually for two reasons, since June 2024 I think. This commit uses the wrong path for
Entlitlements.plist:The actual path is ${CMAKE_SOURCE_DIR}/resources/dist/macos/Entitlements.plist
Also this entire block depends on
IMHEX_RESIGN_BUNDLEbeing set but nothing sets that. This isn't a problem on GitHub actions build because GitHub actions seem to manually call the codesign utilityImplementation description
Additional things
If this change is merged, will this remove the need for an additional code signing step in the GitHub actions workflow? I think it would just be able to set
-DIMHEX_RESIGN_BUNDLE=ONand be done with it, but I'm not sure how to test GitHub actions to confirm