From aa9c98b6a2db58c8097a81e8074496d2c1c790d1 Mon Sep 17 00:00:00 2001 From: Massimiliano Culpo Date: Tue, 5 May 2020 20:22:32 +0200 Subject: Make usage example consistent with current MV variants (#16468) Since #9481 Python's None is not permitted as a value for MV variants. The string 'none' is used instead. Add the same fix for the amgx and lammps packages --- var/spack/repos/builtin/packages/amgx/package.py | 2 +- var/spack/repos/builtin/packages/lammps/package.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) (limited to 'var') diff --git a/var/spack/repos/builtin/packages/amgx/package.py b/var/spack/repos/builtin/packages/amgx/package.py index ece1d67ee2..c03f3599df 100644 --- a/var/spack/repos/builtin/packages/amgx/package.py +++ b/var/spack/repos/builtin/packages/amgx/package.py @@ -42,7 +42,7 @@ class Amgx(CMakePackage, CudaPackage): if '+cuda' in self.spec: args.append('-DWITH_CUDA=ON') cuda_arch = self.spec.variants['cuda_arch'].value - if cuda_arch is not None: + if cuda_arch != 'none': args.append('-DCUDA_ARCH={0}'.format(cuda_arch[0])) else: args.append('-DWITH_CUDA=OFF') diff --git a/var/spack/repos/builtin/packages/lammps/package.py b/var/spack/repos/builtin/packages/lammps/package.py index 2320cf0050..5bf2986858 100644 --- a/var/spack/repos/builtin/packages/lammps/package.py +++ b/var/spack/repos/builtin/packages/lammps/package.py @@ -150,7 +150,7 @@ class Lammps(CMakePackage, CudaPackage): args.append('-DPKG_GPU=ON') args.append('-DGPU_API=cuda') cuda_arch = spec.variants['cuda_arch'].value - if cuda_arch is not None: + if cuda_arch != 'none': args.append('-DGPU_ARCH=sm_{0}'.format(cuda_arch[0])) args.append('-DCUDA_MPS_SUPPORT={0}'.format( 'ON' if '+cuda_mps' in spec else 'OFF')) -- cgit v1.2.3-70-g09d2