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

Unexpected rpaths on macOS #199

Closed
meshula opened this issue Aug 6, 2016 · 1 comment
Closed

Unexpected rpaths on macOS #199

meshula opened this issue Aug 6, 2016 · 1 comment
Labels
Build A problem with building or installing the library.

Comments

@meshula
Copy link
Contributor

meshula commented Aug 6, 2016

See some samples below. Cmake generates dylibs with a 0.0, and symlinks to versions without the 0.0. Is it expected that the rpaths point to symlinks instead of the library itself? Is the reasoning that iterations at the 12.x.y level are irrelevant, so we can keep building OpenExr minor versions but don't need to relink dependencies since the rpath will resolve through a symlink?

local/lib/libHalf.12.0.0.dylib:
name @rpath/libHalf.12.dylib (offset 24)

local/lib/libIex-2_2.12.0.0.dylib:
name @rpath/libIex-2_2.12.dylib (offset 24)

otool -l local/lib/libIlmImf-2_2.22.0.0.dylib|grep dylib ~/Projects/usd2/stage
local/lib/libIlmImf-2_2.22.0.0.dylib:
name @rpath/libIlmImf-2_2.22.dylib (offset 24)
name @rpath/libHalf.12.dylib (offset 24)
name @rpath/libIex-2_2.12.dylib (offset 24)
name @rpath/libImath-2_2.12.dylib (offset 24)
name @rpath/libIlmThread-2_2.12.dylib (offset 24)

@cary-ilm cary-ilm added Build A problem with building or installing the library. and removed Mac OS labels Jun 29, 2019
@cary-ilm cary-ilm added this to the Needs Attention milestone Jun 29, 2019
@kdt3rd
Copy link
Contributor

kdt3rd commented Jul 21, 2019

Closing old ticket...

All of the library names should now be centralized in the newly refactored cmake coming in the next release. The so version (22.0.0) [as different from the library version] should indicate the abi present, and so yes, does not require the minor / patch releases in there, until we were to do such a release, which I don't think we've done yet :)

@kdt3rd kdt3rd closed this as completed Jul 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build A problem with building or installing the library.
Projects
None yet
Development

No branches or pull requests

4 participants