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

Automatically package OpenSSL inside the framework #668

Merged
merged 4 commits into from
Jan 30, 2019

Conversation

tiennou
Copy link
Contributor

@tiennou tiennou commented Sep 27, 2018

As per the script's documentation :

This script looks up an executable's list of shared libraries, copies
non-standard ones (ie. anything not under /usr or /System/) into the target's
bundle and updates the executable install_name to point to the "packaged"
version.

In short, it will grab the Homebrew version of OpenSSL dylibs we're using and fixup the ObjectiveGit framework to use a copy of them.

It's 2018, that install_name should have been automated by now 😉.

Ref #667.

@tiennou
Copy link
Contributor Author

tiennou commented Sep 27, 2018

@pietbrauer Do you/GH have an opinion on this ? Just want to make sure I'm not stomping over some build system of yours by doing that…

@mxcl
Copy link

mxcl commented Sep 28, 2018

I had to edit the script to get it to build with Carthage.

deps = `otool -arch x86_64 -L #{TARGET_EXECUTABLE_PATH}`

Since for some reason the CURRENT_ARCH variable was set to undefined_arch. Might be an Xcode 10 issue.

@tiennou tiennou force-pushed the fix/openssl-linkpath branch 2 times, most recently from 5cc8e61 to 61051a5 Compare October 26, 2018 21:12
@tiennou
Copy link
Contributor Author

tiennou commented Oct 27, 2018

Okay, I've rejigged that script, because I tried to take care of codesigning, which needed refactoring, but ultimately decided that I don't care enough (also codesign --deep seems to be a thing). So this is the refactored version without codesigning.

I've removed the offending CURRENT_ARCH, since it seems to be going away, and AFAIK x86_64 rules the world anyways we only call that on a mono-arch binary.

This automatically copies libssl & libcrypto inside our framework bundle.
@Uncommon
Copy link
Contributor

Any progress on this? It looks like the travis build failed because of a file permissions error.

@Uncommon
Copy link
Contributor

Thanks for the fix @tiennou. Can we please get this merged?

@tiennou tiennou merged commit fd12f0f into libgit2:master Jan 30, 2019
@tiennou tiennou deleted the fix/openssl-linkpath branch January 30, 2019 00:47
@tiennou
Copy link
Contributor Author

tiennou commented Jan 30, 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