diff options
author | Harmen Stoppels <harmenstoppels@gmail.com> | 2022-04-27 11:15:48 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-04-27 11:15:48 +0200 |
commit | f273e7d3294026bcc8ba03310344893f5e5043bb (patch) | |
tree | 8812b84e1b1942c852faa87af68440075ab253a0 | |
parent | 351072cd9fba3b90f00cf8dc6e51c6d7cfd34fd1 (diff) | |
download | spack-f273e7d3294026bcc8ba03310344893f5e5043bb.tar.gz spack-f273e7d3294026bcc8ba03310344893f5e5043bb.tar.bz2 spack-f273e7d3294026bcc8ba03310344893f5e5043bb.tar.xz spack-f273e7d3294026bcc8ba03310344893f5e5043bb.zip |
openmpi: add toggle for rsh plm (#30251)
By default `openmpi` needs `rsh` from `openssh`, which is a somewhat
redundant dependency for clusters using slurm. This PR adds a toggle to
allow users to disable the ssh/rsh plm altogether.
-rw-r--r-- | var/spack/repos/builtin/packages/openmpi/package.py | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/var/spack/repos/builtin/packages/openmpi/package.py b/var/spack/repos/builtin/packages/openmpi/package.py index f5065a1847..6cf080cc0b 100644 --- a/var/spack/repos/builtin/packages/openmpi/package.py +++ b/var/spack/repos/builtin/packages/openmpi/package.py @@ -246,6 +246,7 @@ class Openmpi(AutotoolsPackage, CudaPackage): variant('lustre', default=False, description="Lustre filesystem library support") variant('romio', default=True, description='Enable ROMIO support') + variant('rsh', default=True, description='Enable rsh (openssh) process lifecycle management') # Adding support to build a debug version of OpenMPI that activates # Memchecker, as described here: # @@ -329,7 +330,7 @@ class Openmpi(AutotoolsPackage, CudaPackage): depends_on('pmix', when='+pmix') depends_on('pmix@3.2:', when='@4.0:4 +pmix') - depends_on('openssh', type='run') + depends_on('openssh', type='run', when='+rsh') conflicts('+cxx_exceptions', when='%nvhpc', msg='nvc does not ignore -fexceptions, but errors') @@ -665,6 +666,10 @@ class Openmpi(AutotoolsPackage, CudaPackage): # uct btl doesn't work with some UCX versions so just disable config_args.append('--enable-mca-no-build=btl-uct') + # Remove ssh/rsh pml + if spec.satisfies('~rsh'): + config_args.append('--enable-mca-no-build=plm-rsh') + # some scientific packages ignore deprecated/remove symbols. Re-enable # them for now, for discussion see # https://github.com/open-mpi/ompi/issues/6114#issuecomment-446279495 |