search

natefoo / slurm-drmaa

DRMAA for Slurm: Implementation of the DRMAA C bindings for Slurm
GNU General Public License v3.0
48 stars 22 forks source link

Fix for LD_LIBRARY_PATH and libslurm #57

Closed pwablito closed 2 years ago

pwablito commented 2 years ago

This is a simple fix that implements one the solutions @reid-wagner proposed in #44. I was creating a Spack package for slurm-drmaa and found that this should fix the install.

natefoo commented 2 years ago

I am chasing down the actual cause of this on #44 as the behavior varies by platform. That said, if we do end up committing something that explicitly sets the runtime link path, I would prefer that it be with -Wl,-rpath rather than LD_LIBRARY_PATH.

For the meantime, if you are compiling on EL against a libslurm.so that is not in the standard runtime linker path (e.g. /usr/lib64), you can configure with (where /path/to/slurm is the --prefix of the Slurm installation:

LDFLAGS=-Wl,-rpath=/path/to/slurm/lib ./configure --with-slurm-inc=/path/to/slurm/include --with-slurm-lib=/path/to/slurm/lib
natefoo commented 2 years ago

This is superseded by the changes in #62, but thank you for the contribution, sorry for the delay in getting it fixed and merged.