-
Notifications
You must be signed in to change notification settings - Fork 425
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
Binaries may not expect shared libraries in /lib{,64} #4
Comments
Ahh, good find! I think I have a very portable way of fixing this and will implement as soon as possible and let you know for testing. Thanks! |
I updated the ldd dependency resolver to use the default library paths and use LD_LIBRARY_PATH to update the search path. Will you recreate your SAPPs and let me know what you find? Thank you! |
Looks good! |
I closed to early. When an application itself requires LD_LIBRARY_PATH set, the libraries are fetched into the sapp, but they are not found on execution. Am I missing something? |
Ahhh, that is a good catch. Some variables should be caught and replicated into the SAPP. I'll think about how to do it and implement something when I get to the computer. Let me know if you think of a fix too! Thanks!! Sent from my iPhone
|
I just committed a method which will find all libraries in the SAPP and include them in the LD_LIBRARY_PATH. Can you check to see if that is working now? Thanks! |
Hmm, no. ...> singularity build heat.spec What is your way to debug this? |
Looking into ~/.sapp-cache/heat/c/singularity: Newline instead of colon? |
Woops... Sorry about that! I just committed a fix that will work... Will test as soon as I get into work. |
The fix seems to be working for me, but I don't have a use-case on this system with multiple libdirs. Can you confirm the latest commits are working better? Thanks! |
Works fine here. |
Excellent, thanks! |
Restrict /apps access
The real URI to which we are pushing is computed at a lower level, in remote endpoint handling, than we have access to in the CLI newpair code. At present it is only correct for the default keystore. Remove the URI from the message, until we can address the issue by having the CLI code able to know the true correct URI in all cases. Fixes #4 See also #23
The real URI to which we are pushing is computed at a lower level, in remote endpoint handling, than we have access to in the CLI newpair code. At present it is only correct for the default keystore. Remove the URI from the message, until we can address the issue by having the CLI code able to know the true correct URI in all cases. Fixes #4 See also #23
All shared libraries get copied into /lib or /lib64 in the "container".
This does not work with binaries from Ubuntu, as they expect shared libraries (have rpath set) in "/lib/x86_64-linux-gnu/", even for "/bin/sh". Explicitly setting LD_LIBRARY_PATH in src/sexec.c with setenv(2) helps.
The text was updated successfully, but these errors were encountered: