Skip to content

Commit 69fe72b

Browse files
authored
Merge pull request #4099 from Thyre/20260324152539_new_pr_superlu
avoid crash in superlu easyblock if OpenMPI is not a dependency
2 parents 54f5db1 + b4637b6 commit 69fe72b

1 file changed

Lines changed: 7 additions & 6 deletions

File tree

easybuild/easyblocks/s/superlu.py

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -142,12 +142,13 @@ def test_step(self):
142142
# allow oversubscription of number of processes over number of available cores with OpenMPI 3.0 & newer,
143143
# to avoid that some tests fail if only a handful of cores are available
144144
ompi_ver = get_software_version('OpenMPI')
145-
if LooseVersion(ompi_ver) >= LooseVersion('5.0'):
146-
if 'PRTE_MCA_rmaps_default_mapping_policy' not in self.cfg['pretestopts']:
147-
self.cfg.update('pretestopts', "export PRTE_MCA_rmaps_default_mapping_policy=:oversubscribe && ")
148-
elif LooseVersion(ompi_ver) >= LooseVersion('3.0'):
149-
if 'OMPI_MCA_rmaps_base_oversubscribe' not in self.cfg['pretestopts']:
150-
self.cfg.update('pretestopts', "export OMPI_MCA_rmaps_base_oversubscribe=true && ")
145+
if ompi_ver:
146+
if LooseVersion(ompi_ver) >= LooseVersion('5.0'):
147+
if 'PRTE_MCA_rmaps_default_mapping_policy' not in self.cfg['pretestopts']:
148+
self.cfg.update('pretestopts', "export PRTE_MCA_rmaps_default_mapping_policy=:oversubscribe && ")
149+
elif LooseVersion(ompi_ver) >= LooseVersion('3.0'):
150+
if 'OMPI_MCA_rmaps_base_oversubscribe' not in self.cfg['pretestopts']:
151+
self.cfg.update('pretestopts', "export OMPI_MCA_rmaps_base_oversubscribe=true && ")
151152

152153
super().test_step()
153154

0 commit comments

Comments
 (0)