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

RUNPATH/LD_LIBRARY_PATH Fixes #138

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

RUNPATH/LD_LIBRARY_PATH Fixes #138

wants to merge 12 commits into from

Conversation

bholt
Copy link
Member

@bholt bholt commented Jan 23, 2014

Using RUNPATH again, and should be prepending LD_LIBRARY_PATH correctly, but doesn't solve the problem of being able to specify libs like we wanted to do on Pal, because whatever's in LD_LIBRARY_PATH will still be prepended by srun_prolog.

This solution is, first of all, pretty messy as it involves now hardcoding the lib path in the run script, but still doesn't allow us to point at different libraries dynamically.

@nelsonje: Any ideas about how to do this better?

doesn't solve the problem of being able to specify libs, like we wanted
to do on Pal, because whatever's in LD_LIBRARY_PATH will still be
prepended by srun_prolog
@bholt
Copy link
Member Author

bholt commented Jan 23, 2014

Oh, and I also was just looking at how igor_common isolates exes, and I can't see anywhere where I'm actually changing LD_LIBRARY_PATH. So now I'm wondering if it's even been using the copied libraries, or whether it has still been loading them from where they were this whole time...

@bholt
Copy link
Member Author

bholt commented Jan 23, 2014

(btw, I think this need not hold up flipping the bit, it's definitely an edge case)

@bholt
Copy link
Member Author

bholt commented Feb 25, 2014

@nelsonje @bmyerz: I've gotten LD_LIBRARY_PATH working again, with Igor isolation. I also fixed the segfault failure problems I was seeing, so we're now using Glog to generate backtraces, and calling Gasnet suspend if "--freeze-on-error" is specified.

@bholt
Copy link
Member Author

bholt commented Feb 25, 2014

Let me know if you think it's alright to merge into master.

/// called on failures to backtrace and pause for debugger
void failure_function() {
void failure_function(int ignore = 0) {
fprintf(stderr, "!! failure function\n");
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will disappear, right? :-)

@bholt
Copy link
Member Author

bholt commented Mar 9, 2014

Hmm, actually it looks like this didn't completely solve the problem: now Gasnet freeze works for segfaults, but not for assertion failures.

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.

2 participants