package sanity: ensure all variant defaults are allowed values (#20373)

This commit is contained in:
Massimiliano Culpo
2020-12-15 10:22:15 +01:00
committed by Tamara Dahlgren
parent 27f6b3df92
commit bf3a873a42
19 changed files with 36 additions and 25 deletions

View File

@@ -27,7 +27,7 @@ class CbtfArgonavis(CMakePackage):
to point to target build.")
variant('runtime', default=False,
description="build only the runtime libraries and collectors.")
variant('build_type', default='None', values=('None'),
variant('build_type', default='None', values=('None',),
description='CMake build type')
depends_on("cmake@3.0.2:", type='build')

View File

@@ -39,7 +39,7 @@ class CbtfKrell(CMakePackage):
description="Build mpi experiment collector for mpich MPI.")
variant('runtime', default=False,
description="build only the runtime libraries and collectors.")
variant('build_type', default='None', values=('None'),
variant('build_type', default='None', values=('None',),
description='CMake build type')
variant('cti', default=False,
description="Build MRNet with the CTI startup option")

View File

@@ -20,7 +20,7 @@ class CbtfLanl(CMakePackage):
version('1.9.1.1', branch='1.9.1.1')
version('1.9.1.0', branch='1.9.1.0')
variant('build_type', default='None', values=('None'),
variant('build_type', default='None', values=('None',),
description='CMake build type')
variant('runtime', default=False,

View File

@@ -29,7 +29,7 @@ class Cbtf(CMakePackage):
variant('runtime', default=False,
description="build only the runtime libraries and collectors.")
variant('build_type', default='None', values=('None'),
variant('build_type', default='None', values=('None',),
description='CMake build type')
depends_on("cmake@3.0.2:", type='build')

View File

@@ -25,7 +25,7 @@ class Elsi(CMakePackage):
)
variant(
'elpa2_kernel', default="none", description="ELPA2 Kernel",
values=('AVX', 'AVX2', 'AVX512'), multi=False
values=('none', 'AVX', 'AVX2', 'AVX512'), multi=False
)
variant(
'enable_pexsi', default=False, description='Enable PEXSI support'

View File

@@ -36,7 +36,7 @@ class Fairlogger(CMakePackage):
multi=False,
description='CMake build type')
variant('cxxstd', default='default',
values=('11', '14', '17'),
values=('default', '11', '14', '17'),
multi=False,
description='Use the specified C++ standard when building.')
variant('pretty',

View File

@@ -31,7 +31,7 @@ class GpuBurn(MakefilePackage, CudaPackage):
'cuda_arch',
description='CUDA architecture',
default='none',
values=cuda_arch_values,
values=('none',) + cuda_arch_values,
multi=False
)

View File

@@ -67,7 +67,7 @@ class Hdf5(AutotoolsPackage):
variant('pic', default=True,
description='Produce position-independent code (for shared libs)')
# Build HDF5 with API compaitibility.
variant('api', default='none', description='choose api compatibility', values=('v114', 'v112', 'v110', 'v18', 'v16'), multi=False)
variant('api', default='none', description='choose api compatibility', values=('none', 'v114', 'v112', 'v110', 'v18', 'v16'), multi=False)
conflicts('api=v114', when='@1.6:1.12.99', msg='v114 is not compatible with this release')
conflicts('api=v112', when='@1.6:1.10.99', msg='v112 is not compatible with this release')

View File

@@ -30,7 +30,8 @@ class Jube(PythonPackage):
variant(
'resource_manager', default='none',
description='Select resource manager templates',
values=('loadleveler', 'lsf', 'moab', 'pbs', 'slurm'), multi=False
values=('none', 'loadleveler', 'lsf', 'moab', 'pbs', 'slurm'),
multi=False
)
depends_on('py-setuptools', type='build')

View File

@@ -77,9 +77,8 @@ class KokkosLegacy(Package):
'Volta70', 'Volta72')
# C++ standard variant
variant('cxxstd', default='none',
values=('c++11', 'c++14', 'c++17', 'c++1y', 'c++1z', 'c++2a'),
multi=False,
cxx_stds = ('none', 'c++11', 'c++14', 'c++17', 'c++1y', 'c++1z', 'c++2a')
variant('cxxstd', default='none', values=cxx_stds, multi=False,
description='set cxxstandard Kokkos option')
# Host architecture variant

View File

@@ -72,13 +72,13 @@ class Kokkos(CMakePackage, CudaPackage):
'tests': [False, 'Build for tests'],
}
amd_gpu_arches = [
amd_gpu_arches = (
'fiji',
'gfx901',
'vega900',
'vega906',
]
variant("amd_gpu_arch", default='none', values=amd_gpu_arches,
)
variant("amd_gpu_arch", default='none', values=('none',) + amd_gpu_arches,
description="AMD GPU architecture")
conflicts("+hip", when="amd_gpu_arch=none")

View File

@@ -31,7 +31,7 @@ class Libbeagle(AutotoolsPackage, CudaPackage):
'cuda_arch',
description='CUDA architecture',
default='none',
values=cuda_arch_values,
values=('none',) + cuda_arch_values,
multi=False
)
conflicts('cuda_arch=none', when='+cuda',

View File

@@ -51,7 +51,7 @@ class OpenspeedshopUtils(CMakePackage):
variant('cuda', default=False,
description="build with cuda packages included.")
variant('build_type', default='None', values=('None'),
variant('build_type', default='None', values=('None',),
description='CMake build type')
# MPI variants

View File

@@ -46,7 +46,7 @@ class Openspeedshop(CMakePackage):
variant('gui', default='qt3', values=('none', 'qt3', 'qt4'),
description='Build or not build a GUI of choice')
variant('build_type', default='None', values=('None'),
variant('build_type', default='None', values=('None',),
description='CMake build type')
# MPI variants

View File

@@ -29,7 +29,7 @@ class Rr(CMakePackage):
# Only 'Release' is supported at the moment
variant('build_type', default='Release',
description='The build type to build',
values=('Release'))
values=('Release',))
def patch(self):
# because otherwise CMake would try and fail to set RPATH of

View File

@@ -66,7 +66,7 @@ class VtkM(CMakePackage, CudaPackage):
'gfx908': 'vega908'
}
variant('amdgpu_target', default='none', multi=True, values=amdgpu_targets)
variant('amdgpu_target', default='none', multi=True, values=('none',) + amdgpu_targets)
conflicts("+hip", when="amdgpu_target=none")
depends_on("cmake@3.12:", type="build") # CMake >= 3.12