zhangyiss/spack
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

openmpi: allow to build with ^slurm ~pmi for >3.0.0 with PMIx (#22086)

Browse Source
This commit is contained in:
Andrew W Elble 2021-04-06 03:40:33 -04:00 committed by GitHub
parent 023524365c
commit a57f56c5b9
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
var/spack/repos/builtin/packages/openmpi/package.py
View File

@ -337,7 +337,7 @@ class Openmpi(AutotoolsPackage):
# knem support was added in 1.5 # knem support was added in 1.5
conflicts('fabrics=knem', when='@:1.4') conflicts('fabrics=knem', when='@:1.4')
conflicts('schedulers=slurm ~pmi', when='@1.5.4:', conflicts('schedulers=slurm ~pmi', when='@1.5.4:2.999.999',
msg='+pmi is required for openmpi(>=1.5.5) to work with SLURM.') msg='+pmi is required for openmpi(>=1.5.5) to work with SLURM.')
conflicts('schedulers=loadleveler', when='@3.0.0:', conflicts('schedulers=loadleveler', when='@3.0.0:',
msg='The loadleveler scheduler is not supported with ' msg='The loadleveler scheduler is not supported with '
@ -631,7 +631,11 @@ def configure_args(self):
# for versions older than 3.0.3,3.1.3,4.0.0 # for versions older than 3.0.3,3.1.3,4.0.0
# Presumably future versions after 11/2018 should support slurm+static # Presumably future versions after 11/2018 should support slurm+static
if spec.satisfies('schedulers=slurm'): if spec.satisfies('schedulers=slurm'):
config_args.append('--with-pmi={0}'.format(spec['slurm'].prefix)) if spec.satisfies('+pmi'):
config_args.append('--with-pmi={0}'.format(
spec['slurm'].prefix))
else:
config_args.extend(self.with_or_without('pmi'))
if spec.satisfies('@3.1.3:') or spec.satisfies('@3.0.3'): if spec.satisfies('@3.1.3:') or spec.satisfies('@3.0.3'):
if '+static' in spec: if '+static' in spec:
config_args.append('--enable-static') config_args.append('--enable-static')