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.

Sign up for GitHub

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

Jump to bottom

LLVM 3.7 dylib not loaded on OS X nightly #17484

Closed
ararslan opened this issue Jul 18, 2016 · 2 comments
Closed

LLVM 3.7 dylib not loaded on OS X nightly #17484

ararslan opened this issue Jul 18, 2016 · 2 comments

Comments

@ararslan
Copy link
Member

I've seen this on the JuliaStats Travis builds today:

$ julia --check-bounds=yes -e 'Pkg.clone(pwd()); Pkg.build("DataFrames"); Pkg.test("DataFrames"; coverage=true)'
dyld: Library not loaded: @rpath/libLLVM-3.7.dylib
  Referenced from: /Users/travis/julia/bin/julia
  Reason: image not found
/Users/travis/build.sh: line 45:   472 Trace/BPT trap: 5       julia --check-bounds=yes -e 'Pkg.clone(pwd()); Pkg.build("DataFrames"); Pkg.test("DataFrames"; coverage=true)'

It's causing the builds to fail, but only on OS X nightly. If I remember correctly, @Keno mentioned something recently about rpath getting changed?
@Keno
Copy link
Member

Keno commented Jul 18, 2016

Which commit was that version of julia built from?

@Keno
Copy link
Member

Keno commented Jul 18, 2016

Ok, yeah, the latest nightly has that problem. Will take a look.

Keno added a commit that referenced this issue Jul 18, 2016
@Keno
Don't introduce spurious libLLVM dependency into julia binary
794b99e 
One linux the linker realizes it's not required and doesn't note down
the dependency, but on OS X it is included anyway. This causes problems
with the new RPATH handling since we now omit lib/julia from the
executable's RPATH. Fix this, by not forcing the executable to link
against libLLVM.

Fixes #17484.
Keno added a commit that referenced this issue Jul 18, 2016
@Keno
Don't introduce spurious libLLVM dependency into julia binary
cc069bd 
This causes problems with the new RPATH handling since we now omit
lib/julia from the executable's RPATH. Fix this by not forcing
the executable to link against libLLVM, since we don't require it.

Fixes #17484.
@Keno Keno mentioned this issue Jul 18, 2016
Merged
@ranjanan ranjanan mentioned this issue Jul 19, 2016
Merged
@ararslan ararslan mentioned this issue Jul 19, 2016
Merged
tkelman pushed a commit that referenced this issue Jul 19, 2016
@Keno @tkelman
Don't introduce spurious libLLVM dependency into julia binary (#17486)
c7ca319 
* Don't introduce spurious libLLVM dependency into julia binary

This causes problems with the new RPATH handling since we now omit
lib/julia from the executable's RPATH. Fix this by not forcing
the executable to link against libLLVM, since we don't require it.

Fixes #17484.

* Provide link-time rpath

Unlike ld.so, ld does not understand `$ORIGIN`, so provide the full path to it at link
time to make sure it can find the appropriate dependencies.
@ararslan ararslan mentioned this issue Jul 19, 2016
Merged
mfasi pushed a commit to mfasi/julia that referenced this issue Sep 5, 2016
@Keno @mfasi
Don't introduce spurious libLLVM dependency into julia binary (JuliaL…
17bd99a 
…ang#17486)

* Don't introduce spurious libLLVM dependency into julia binary

This causes problems with the new RPATH handling since we now omit
lib/julia from the executable's RPATH. Fix this by not forcing
the executable to link against libLLVM, since we don't require it.

Fixes JuliaLang#17484.

* Provide link-time rpath

Unlike ld.so, ld does not understand `$ORIGIN`, so provide the full path to it at link
time to make sure it can find the appropriate dependencies.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Keno @ararslan