hlrs-software-stack/spack
7
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Fix openmpi to work with slurm/pmi and mxm (#8427)

Browse source 
* Add binutils+libiberty for mxm fabric
* Help find the pmi library when using the slurm scheduler
This commit is contained in:
Tom Merrick 2018-08-02 13:06:39 -05:00 committed by scheibelp
parent c94109c34c
commit 51aa604ed2
1 changed files with 6 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
var/spack/repos/builtin/packages/openmpi/package.py
View file

@ -242,6 +242,8 @@ class Openmpi(AutotoolsPackage):
depends_on('valgrind~mpi', when='+memchecker')
depends_on('ucx', when='fabrics=ucx')
depends_on('libfabric', when='fabrics=libfabric')
depends_on('slurm', when='schedulers=slurm')
depends_on('binutils+libiberty', when='fabrics=mxm')
conflicts('+cuda', when='@:1.6') # CUDA support was added in 1.7
conflicts('fabrics=psm2', when='@:1.8') # PSM2 support was added in 1.10.0
@ -354,8 +356,11 @@ def configure_args(self):
# https://github.com/open-mpi/ompi/issues/4338#issuecomment-383982008
#
# adding --enable-static silently disables slurm support via pmi/pmi2
if not spec.satisfies('schedulers=slurm'):
if spec.satisfies('schedulers=slurm'):
config_args.append('--with-pmi={0}'.format(spec['slurm'].prefix))
else:
config_args.append('--enable-static')
config_args.extend(self.with_or_without('pmi'))
if spec.satisfies('@2.0:'):
# for Open-MPI 2.0:, C++ bindings are disabled by default.
@ -368,8 +373,6 @@ def configure_args(self):
config_args.extend(self.with_or_without('fabrics'))
# Schedulers
config_args.extend(self.with_or_without('schedulers'))
# PMI
config_args.extend(self.with_or_without('pmi'))
config_args.extend(self.enable_or_disable('memchecker'))
if spec.satisfies('+memchecker', strict=True):