Skip to content
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

Add FAKESIGN_IPA option #77

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add FAKESIGN_IPA option #77

wants to merge 2 commits into from

Conversation

opa334
Copy link

@opa334 opa334 commented Sep 10, 2022

Preserves entitlements of all binaries and fakesigns everything using ldid.

Perfect for installation using TrollStore.

(install_name_tool also needs the --no-strip-codesig flag because otherwise the entitlements are removed by it)

@Al4ise
Copy link

Al4ise commented Oct 19, 2022

Insert_dylib with --no-strip-codesig works in 50% of cases at max. I've personally had many issues with it. A better solution is to dump entitlements in a separate file, strip codesign, then install_name_tool, and then restore the entitlements

@kabiroberai
Copy link
Owner

Excuse the late response, but thanks for the PR! I agree that dumping and restoring ents might be a better idea.

For maximum flexibility, I think the way to go here would be to 1) allow PROFILE = - to indicate ad-hoc signing, and 2) allow a custom file to be set for ENTITLEMENTS — which is something I should've honestly done in the first place, since the profile ents don't always map 1:1 with the ents the ipa should have. Also, for consistency, we can use codesign -fs - for ad-hoc signing rather than ldid since atm Theos Jailed is macOS-only anyway.

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