Skip to content

Commit

Permalink
Don't consider lib/release for impi >= 2021.11
Browse files Browse the repository at this point in the history
lib/release is a symbolic link to lib, so we no longer need to
include it in LD_LIBRARY_PATH and LIBRARY_PATH
  • Loading branch information
bartoldeman committed Jun 27, 2024
1 parent 4c0f0fe commit a60c47d
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions easybuild/easyblocks/i/impi.py
Original file line number Diff line number Diff line change
Expand Up @@ -211,7 +211,9 @@ def sanity_check_step(self):
mpi_subdir = self.get_versioned_subdir('mpi')
bin_dir = os.path.join(mpi_subdir, 'bin')
include_dir = os.path.join(mpi_subdir, 'include')
lib_dir = os.path.join(mpi_subdir, 'lib', 'release')
lib_dir = os.path.join(mpi_subdir, 'lib')
if impi_ver < LooseVersion('2021.11'):
lib_dir = os.path.join(lib_dir, 'release')

elif impi_ver >= LooseVersion('2019'):
bin_dir = os.path.join('intel64', 'bin')
Expand Down Expand Up @@ -288,9 +290,10 @@ def make_module_req_guess(self):
mpi_subdir = self.get_versioned_subdir('mpi')
lib_dirs = [
os.path.join(mpi_subdir, 'lib'),
os.path.join(mpi_subdir, 'lib', 'release'),
os.path.join(mpi_subdir, 'libfabric', 'lib'),
]
if impi_ver < LooseVersion('2021.11'):
lib_dirs.insert(1, os.path.join(mpi_subdir, 'lib', 'release'))
include_dirs = [os.path.join(mpi_subdir, 'include')]
path_dirs = [
os.path.join(mpi_subdir, 'bin'),
Expand Down

0 comments on commit a60c47d

Please sign in to comment.