diff options
| author | Massimiliano Culpo <massimiliano.culpo@gmail.com> | 2018-05-04 16:27:54 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-05-04 16:27:54 +0200 |
| commit | 0570b075d4793f0e137f70d9ff741c95bd6d9053 (patch) | |
| tree | f85a701fe6152094631d54bb329e7b96cfabaf59 | |
| parent | 593ad23a9f4f51e3ff4231a894ba1998e212fdc4 (diff) | |
| download | spack-0570b075d4793f0e137f70d9ff741c95bd6d9053.tar.gz spack-0570b075d4793f0e137f70d9ff741c95bd6d9053.tar.bz2 spack-0570b075d4793f0e137f70d9ff741c95bd6d9053.tar.xz spack-0570b075d4793f0e137f70d9ff741c95bd6d9053.zip | |
openmpi: don't build static libraries if slurm is activated (#8021)
According to this comment:
https://github.com/open-mpi/ompi/issues/4338#issuecomment-383982008
on an OpenMPI issue, using `--enable-static` silently disable slurm
support.
| -rw-r--r-- | var/spack/repos/builtin/packages/openmpi/package.py | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/var/spack/repos/builtin/packages/openmpi/package.py b/var/spack/repos/builtin/packages/openmpi/package.py index abd1fdde6e..9731558eb0 100644 --- a/var/spack/repos/builtin/packages/openmpi/package.py +++ b/var/spack/repos/builtin/packages/openmpi/package.py @@ -336,8 +336,16 @@ class Openmpi(AutotoolsPackage): spec = self.spec config_args = [ '--enable-shared', - '--enable-static' ] + + # According to this comment on github: + # + # 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'): + config_args.append('--enable-static') + if spec.satisfies('@2.0:'): # for Open-MPI 2.0:, C++ bindings are disabled by default. config_args.extend(['--enable-mpi-cxx']) |