Skip to content

Automatically package OpenSSL inside the framework#668

Merged
tiennou merged 4 commits into
libgit2:masterfrom
tiennou:fix/openssl-linkpath
Jan 30, 2019
Merged

Automatically package OpenSSL inside the framework#668
tiennou merged 4 commits into
libgit2:masterfrom
tiennou:fix/openssl-linkpath

Conversation

@tiennou

@tiennou tiennou commented Sep 27, 2018

Copy link
Copy Markdown
Contributor

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

tiennou commented Sep 27, 2018

Copy link
Copy Markdown
Contributor Author

@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

mxcl commented Sep 28, 2018

Copy link
Copy Markdown

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

tiennou commented Oct 27, 2018

Copy link
Copy Markdown
Contributor Author

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.
@tiennou tiennou force-pushed the fix/openssl-linkpath branch from 61051a5 to ce59040 Compare October 27, 2018 12:35
@Uncommon

Copy link
Copy Markdown
Contributor

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

@Uncommon

Copy link
Copy Markdown
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

tiennou commented Jan 30, 2019

Copy link
Copy Markdown
Contributor Author

🎉

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