From d84c6ad29eb0ff1de78953b3b2c356b793aa306c Mon Sep 17 00:00:00 2001 From: Massimiliano Culpo Date: Sat, 18 Mar 2023 16:39:04 +0100 Subject: cmake build system: make "generator" a variant (#35552) --- var/spack/repos/builtin.mock/packages/gmake/package.py | 15 +++++++++++++++ var/spack/repos/builtin/packages/3dtk/package.py | 3 +-- var/spack/repos/builtin/packages/aluminum/package.py | 3 +-- var/spack/repos/builtin/packages/archer/package.py | 2 +- var/spack/repos/builtin/packages/cepgen/package.py | 5 ++--- var/spack/repos/builtin/packages/cpuinfo/package.py | 4 +--- var/spack/repos/builtin/packages/dbcsr/package.py | 2 +- var/spack/repos/builtin/packages/dd4hep/package.py | 3 +-- var/spack/repos/builtin/packages/dealii/package.py | 3 +-- var/spack/repos/builtin/packages/dihydrogen/package.py | 3 +-- var/spack/repos/builtin/packages/fairlogger/package.py | 1 - var/spack/repos/builtin/packages/fbgemm/package.py | 4 +--- var/spack/repos/builtin/packages/foundationdb/package.py | 3 +-- var/spack/repos/builtin/packages/fp16/package.py | 4 +--- var/spack/repos/builtin/packages/fxdiv/package.py | 4 +--- var/spack/repos/builtin/packages/gdal/package.py | 4 +--- var/spack/repos/builtin/packages/geos/package.py | 4 +--- var/spack/repos/builtin/packages/gloo/package.py | 5 +---- var/spack/repos/builtin/packages/gplates/package.py | 3 +-- var/spack/repos/builtin/packages/halide/package.py | 3 +-- var/spack/repos/builtin/packages/hpx/package.py | 3 +-- var/spack/repos/builtin/packages/hydrogen/package.py | 3 +-- var/spack/repos/builtin/packages/kineto/package.py | 4 +--- var/spack/repos/builtin/packages/lbann/package.py | 3 +-- var/spack/repos/builtin/packages/llvm-amdgpu/package.py | 3 +-- var/spack/repos/builtin/packages/llvm-doe/package.py | 3 +-- .../repos/builtin/packages/llvm-openmp-ompt/package.py | 2 +- var/spack/repos/builtin/packages/llvm/package.py | 3 +-- var/spack/repos/builtin/packages/mxnet/package.py | 3 +-- var/spack/repos/builtin/packages/nnpack/package.py | 4 +--- var/spack/repos/builtin/packages/onnx/package.py | 4 +--- var/spack/repos/builtin/packages/paraview/package.py | 13 ++++--------- .../repos/builtin/packages/pika-algorithms/package.py | 3 +-- var/spack/repos/builtin/packages/pika/package.py | 3 +-- var/spack/repos/builtin/packages/psimd/package.py | 4 +--- var/spack/repos/builtin/packages/pthreadpool/package.py | 4 +--- .../repos/builtin/packages/py-onnx-runtime/package.py | 3 +-- .../repos/builtin/packages/py-onnxruntime/package.py | 3 +-- var/spack/repos/builtin/packages/py-pybind11/package.py | 2 +- var/spack/repos/builtin/packages/qnnpack/package.py | 4 +--- var/spack/repos/builtin/packages/qt-base/package.py | 4 +--- var/spack/repos/builtin/packages/sleef/package.py | 4 +--- var/spack/repos/builtin/packages/spectre/package.py | 3 +-- var/spack/repos/builtin/packages/tensorpipe/package.py | 4 +--- var/spack/repos/builtin/packages/tiramisu/package.py | 4 +--- var/spack/repos/builtin/packages/visit/package.py | 3 +-- var/spack/repos/builtin/packages/xgboost/package.py | 3 +-- var/spack/repos/builtin/packages/xnnpack/package.py | 4 +--- 48 files changed, 65 insertions(+), 116 deletions(-) create mode 100644 var/spack/repos/builtin.mock/packages/gmake/package.py (limited to 'var') diff --git a/var/spack/repos/builtin.mock/packages/gmake/package.py b/var/spack/repos/builtin.mock/packages/gmake/package.py new file mode 100644 index 0000000000..aa5dd8452b --- /dev/null +++ b/var/spack/repos/builtin.mock/packages/gmake/package.py @@ -0,0 +1,15 @@ +# Copyright 2013-2023 Lawrence Livermore National Security, LLC and other +# Spack Project Developers. See the top-level COPYRIGHT file for details. +# +# SPDX-License-Identifier: (Apache-2.0 OR MIT) + +from spack.package import * + + +class Gmake(Package): + """Dummy GMake Package""" + + homepage = "https://www.gnu.org/software/make" + url = "https://ftpmirror.gnu.org/make/make-4.4.tar.gz" + + version("4.4", sha256="ce35865411f0490368a8fc383f29071de6690cbadc27704734978221f25e2bed") diff --git a/var/spack/repos/builtin/packages/3dtk/package.py b/var/spack/repos/builtin/packages/3dtk/package.py index dd1c5e3972..c01faaaba6 100644 --- a/var/spack/repos/builtin/packages/3dtk/package.py +++ b/var/spack/repos/builtin/packages/3dtk/package.py @@ -42,11 +42,10 @@ class _3dtk(CMakePackage): conflicts("~opencv", when="platform=darwin") conflicts("+compact_octree", when="~opengl") - generator = "Ninja" + generator("ninja") depends_on("cmake@3.5:", when="@trunk", type="build") depends_on("cmake@2.6.1:2", when="@1.2", type="build") - depends_on("ninja", type="build") depends_on( "boost@:1.75+serialization+graph+regex+filesystem+system+thread+date_time+program_options" ) diff --git a/var/spack/repos/builtin/packages/aluminum/package.py b/var/spack/repos/builtin/packages/aluminum/package.py index 8946146ac4..0952159617 100644 --- a/var/spack/repos/builtin/packages/aluminum/package.py +++ b/var/spack/repos/builtin/packages/aluminum/package.py @@ -145,8 +145,7 @@ class Aluminum(CMakePackage, CudaPackage, ROCmPackage): conflicts("+cuda", when="+rocm", msg="CUDA and ROCm support are mutually exclusive") conflicts("+nccl", when="+rccl", msg="NCCL and RCCL support are mutually exclusive") - generator = "Ninja" - depends_on("ninja", type="build") + generator("ninja") def cmake_args(self): spec = self.spec diff --git a/var/spack/repos/builtin/packages/archer/package.py b/var/spack/repos/builtin/packages/archer/package.py index 428c37fe0f..4c53949f25 100644 --- a/var/spack/repos/builtin/packages/archer/package.py +++ b/var/spack/repos/builtin/packages/archer/package.py @@ -27,7 +27,7 @@ class Archer(CMakePackage): depends_on("ninja@1.5:", type="build") depends_on("llvm-openmp-ompt@tr6_forwards") - generator = "Ninja" + generator("ninja") def patch(self): if self.spec.satisfies("^llvm@8.0.0:"): diff --git a/var/spack/repos/builtin/packages/cepgen/package.py b/var/spack/repos/builtin/packages/cepgen/package.py index f08ebb49cc..f3268dbb78 100644 --- a/var/spack/repos/builtin/packages/cepgen/package.py +++ b/var/spack/repos/builtin/packages/cepgen/package.py @@ -11,7 +11,6 @@ class Cepgen(CMakePackage): homepage = "https://cepgen.hepforge.org/" url = "https://github.com/cepgen/cepgen/archive/refs/tags/1.0.2patch1.tar.gz" - generator = "Ninja" tags = ["hep"] @@ -19,6 +18,8 @@ class Cepgen(CMakePackage): "1.0.2patch1", sha256="333bba0cb1965a98dec127e00c150eab1a515cd348a90f7b1d66d5cd8d206d21" ) + generator("ninja") + depends_on("gsl") depends_on("openblas") depends_on("hepmc") @@ -26,5 +27,3 @@ class Cepgen(CMakePackage): depends_on("lhapdf") depends_on("pythia6") depends_on("root") - - depends_on("ninja", type="build") diff --git a/var/spack/repos/builtin/packages/cpuinfo/package.py b/var/spack/repos/builtin/packages/cpuinfo/package.py index 7531dcee98..73285b2e2e 100644 --- a/var/spack/repos/builtin/packages/cpuinfo/package.py +++ b/var/spack/repos/builtin/packages/cpuinfo/package.py @@ -24,10 +24,8 @@ class Cpuinfo(CMakePackage): version("2018-05-13", commit="1e6c8c99d27f2b5eb9d2e6231055c6a4115b85e5") # py-torch@0.4.1 version("2018-04-04", commit="831dc28341b5f20d13e840caf87eaba644d82643") # py-torch@:0.4.0 + generator("ninja") depends_on("cmake@3.5:", type="build") - depends_on("ninja", type="build") - - generator = "Ninja" def cmake_args(self): return [ diff --git a/var/spack/repos/builtin/packages/dbcsr/package.py b/var/spack/repos/builtin/packages/dbcsr/package.py index 0da4477fad..df0a6035e9 100644 --- a/var/spack/repos/builtin/packages/dbcsr/package.py +++ b/var/spack/repos/builtin/packages/dbcsr/package.py @@ -92,7 +92,7 @@ class Dbcsr(CMakePackage, CudaPackage, ROCmPackage): conflicts("smm=blas", when="+opencl") - generator = "Ninja" + generator("ninja") depends_on("ninja@1.10:", type="build") def cmake_args(self): diff --git a/var/spack/repos/builtin/packages/dd4hep/package.py b/var/spack/repos/builtin/packages/dd4hep/package.py index 3930756497..42f51f2801 100644 --- a/var/spack/repos/builtin/packages/dd4hep/package.py +++ b/var/spack/repos/builtin/packages/dd4hep/package.py @@ -101,7 +101,7 @@ class Dd4hep(CMakePackage): deprecated=True, ) - generator = "Ninja" + generator("ninja") # Workarounds for various TBB issues in DD4hep v1.11 # See https://github.com/AIDASoft/DD4hep/pull/613 . @@ -137,7 +137,6 @@ class Dd4hep(CMakePackage): ) depends_on("cmake @3.12:", type="build") - depends_on("ninja", type="build") depends_on("boost @1.49:") depends_on("boost +iostreams", when="+ddg4") depends_on("boost +system +filesystem", when="%gcc@:7") diff --git a/var/spack/repos/builtin/packages/dealii/package.py b/var/spack/repos/builtin/packages/dealii/package.py index 0aa6b35a3c..3030f84142 100644 --- a/var/spack/repos/builtin/packages/dealii/package.py +++ b/var/spack/repos/builtin/packages/dealii/package.py @@ -23,7 +23,7 @@ class Dealii(CMakePackage, CudaPackage): # only add for immediate deps. transitive_rpaths = False - generator = "Ninja" + generator("ninja") version("master", branch="master") version("9.4.0", sha256="238677006cd9173658e5b69cdd1861f800556982db6005a3cc5eb8329cc1e36c") @@ -140,7 +140,6 @@ class Dealii(CMakePackage, CudaPackage): # See https://github.com/spack/spack/pull/22303 for reference depends_on(Boost.with_default_variants) depends_on("lapack") - depends_on("ninja", type="build") depends_on("suite-sparse") depends_on("zlib") diff --git a/var/spack/repos/builtin/packages/dihydrogen/package.py b/var/spack/repos/builtin/packages/dihydrogen/package.py index 67c5544525..ca53a89765 100644 --- a/var/spack/repos/builtin/packages/dihydrogen/package.py +++ b/var/spack/repos/builtin/packages/dihydrogen/package.py @@ -116,8 +116,7 @@ class Dihydrogen(CMakePackage, CudaPackage, ROCmPackage): depends_on("half", when="+half") - generator = "Ninja" - depends_on("ninja", type="build") + generator("ninja") depends_on("cmake@3.17.0:", type="build") depends_on("spdlog", when="@:0.1,0.2:") diff --git a/var/spack/repos/builtin/packages/fairlogger/package.py b/var/spack/repos/builtin/packages/fairlogger/package.py index c22c2f8c65..bce80c7755 100644 --- a/var/spack/repos/builtin/packages/fairlogger/package.py +++ b/var/spack/repos/builtin/packages/fairlogger/package.py @@ -16,7 +16,6 @@ class Fairlogger(CMakePackage): url = "https://github.com/FairRootGroup/FairLogger/archive/v1.2.0.tar.gz" git = "https://github.com/FairRootGroup/FairLogger.git" maintainers("dennisklein", "ChristianTackeGSI") - # generator = 'Ninja' version("develop", branch="dev", get_full_repo=True) version("1.9.0", sha256="13bcaa0d4129f8d4e69a0a2ece8e5b7073760082c8aa028e3fc0c11106503095") diff --git a/var/spack/repos/builtin/packages/fbgemm/package.py b/var/spack/repos/builtin/packages/fbgemm/package.py index 176efcbce0..3b853465c5 100644 --- a/var/spack/repos/builtin/packages/fbgemm/package.py +++ b/var/spack/repos/builtin/packages/fbgemm/package.py @@ -54,15 +54,13 @@ class Fbgemm(CMakePackage): "2018-12-04", commit="0d5a159b944252e70a677236b570f291943e0543", submodules=True ) # py-torch@1.0.0 + generator("ninja") depends_on("cmake@3.5:", type="build") - depends_on("ninja", type="build") depends_on("python", type="build") depends_on("llvm-openmp", when="%apple-clang") conflicts("%gcc@:4", msg="FBGEMM requires GCC 5+") - generator = "Ninja" - @run_before("cmake") def check_requirements(self): if "avx2" not in self.spec.target: diff --git a/var/spack/repos/builtin/packages/foundationdb/package.py b/var/spack/repos/builtin/packages/foundationdb/package.py index 64f2125c19..61754d6781 100644 --- a/var/spack/repos/builtin/packages/foundationdb/package.py +++ b/var/spack/repos/builtin/packages/foundationdb/package.py @@ -30,8 +30,7 @@ class Foundationdb(CMakePackage): # for instance depends_on('boost +filesystem') # See https://github.com/spack/spack/pull/22303 for reference depends_on(Boost.with_default_variants) - generator = "Ninja" - depends_on("ninja", type="build") + generator("ninja") def cmake_args(self): args = ["-DUSE_WERROR=ON"] diff --git a/var/spack/repos/builtin/packages/fp16/package.py b/var/spack/repos/builtin/packages/fp16/package.py index 131e26c033..f3d535efbd 100644 --- a/var/spack/repos/builtin/packages/fp16/package.py +++ b/var/spack/repos/builtin/packages/fp16/package.py @@ -19,10 +19,8 @@ class Fp16(CMakePackage): version("2018-10-10", commit="34d4bf01bbf7376f2baa71b8fa148b18524d45cf") # py-torch@1.0 version("2018-02-25", commit="43d6d17df48ebf622587e7ed9472ea76573799b9") # py-torch@:0.4 + generator("ninja") depends_on("cmake@2.8.12:", type="build") - depends_on("ninja", type="build") - - generator = "Ninja" resource( name="psimd", diff --git a/var/spack/repos/builtin/packages/fxdiv/package.py b/var/spack/repos/builtin/packages/fxdiv/package.py index 6f5ce78dc3..454b8f4b5b 100644 --- a/var/spack/repos/builtin/packages/fxdiv/package.py +++ b/var/spack/repos/builtin/packages/fxdiv/package.py @@ -17,12 +17,10 @@ class Fxdiv(CMakePackage): version("2018-11-16", commit="b742d1143724d646cd0f914646f1240eacf5bd73") # py-torch@1.0:1.5 version("2018-02-24", commit="811b482bcd9e8d98ad80c6c78d5302bb830184b0") # py-torch@0.4 + generator("ninja") depends_on("cmake@3.5:", type="build") - depends_on("ninja", type="build") depends_on("python", type="build") - generator = "Ninja" - def cmake_args(self): return [ self.define("FXDIV_BUILD_TESTS", False), diff --git a/var/spack/repos/builtin/packages/gdal/package.py b/var/spack/repos/builtin/packages/gdal/package.py index ec5fb4adec..dcb90f2305 100644 --- a/var/spack/repos/builtin/packages/gdal/package.py +++ b/var/spack/repos/builtin/packages/gdal/package.py @@ -235,8 +235,8 @@ class Gdal(CMakePackage, AutotoolsPackage, PythonExtension): ) with when("build_system=cmake"): + generator("ninja") depends_on("cmake@3.9:", type="build") - depends_on("ninja", type="build") with when("build_system=autotools"): depends_on("gmake", type="build") @@ -463,8 +463,6 @@ class Gdal(CMakePackage, AutotoolsPackage, PythonExtension): class CMakeBuilder(CMakeBuilder): - generator = "Ninja" - def cmake_args(self): # https://gdal.org/build_hints.html args = [ diff --git a/var/spack/repos/builtin/packages/geos/package.py b/var/spack/repos/builtin/packages/geos/package.py index 10864fed52..03ceaf4772 100644 --- a/var/spack/repos/builtin/packages/geos/package.py +++ b/var/spack/repos/builtin/packages/geos/package.py @@ -51,11 +51,9 @@ class Geos(CMakePackage): version("3.3.4", sha256="cd5400aa5f3fe32246dfed5d238c5017e1808162c865c016480b3e6c07271904") version("3.3.3", sha256="dfcf4bd70ab212a5b7bad21d01b84748f101a545092b56dafdc3882ef3bddec9") + generator("ninja") depends_on("cmake@3.13:", when="@3.10:", type="build") depends_on("cmake@3.8:", type="build") - depends_on("ninja", type="build") - - generator = "Ninja" patch( "https://github.com/libgeos/geos/pull/461.patch?full_index=1", diff --git a/var/spack/repos/builtin/packages/gloo/package.py b/var/spack/repos/builtin/packages/gloo/package.py index bac8531fdb..4ca7d55f43 100644 --- a/var/spack/repos/builtin/packages/gloo/package.py +++ b/var/spack/repos/builtin/packages/gloo/package.py @@ -32,11 +32,8 @@ class Gloo(CMakePackage, CudaPackage): sha256="8e6e9a44e0533ba4303a95a651b1934e5d73632cab08cc7d5a9435e1e64aa424", when="@:2023-01-16", ) - + generator("ninja") depends_on("cmake@2.8.12:", type="build") - depends_on("ninja", type="build") - - generator = "Ninja" def cmake_args(self): return [self.define_from_variant("USE_CUDA", "cuda")] diff --git a/var/spack/repos/builtin/packages/gplates/package.py b/var/spack/repos/builtin/packages/gplates/package.py index 8e33f9caec..ce03344bfa 100644 --- a/var/spack/repos/builtin/packages/gplates/package.py +++ b/var/spack/repos/builtin/packages/gplates/package.py @@ -25,7 +25,6 @@ class Gplates(CMakePackage): depends_on("cmake@3.5:", when="@2.3:", type="build") depends_on("cmake@2.8.8:", when="@2.1", type="build") depends_on("cmake@2.6.2:", when="@2.0", type="build") - depends_on("ninja", type="build") depends_on("gl") depends_on("glu") depends_on("glew") @@ -50,7 +49,7 @@ class Gplates(CMakePackage): # When built in parallel, headers are not generated before they are used # (specifically, ViewportWindowUi.h) with the Makefiles generator. - generator = "Ninja" + generator("ninja") @when("@:2.1") def patch(self): diff --git a/var/spack/repos/builtin/packages/halide/package.py b/var/spack/repos/builtin/packages/halide/package.py index 67ef587b04..f7021ed9ad 100644 --- a/var/spack/repos/builtin/packages/halide/package.py +++ b/var/spack/repos/builtin/packages/halide/package.py @@ -22,7 +22,7 @@ class Halide(CMakePackage, PythonExtension): description="The build type to build", values=("Release", "Debug", "RelWithDebInfo"), ) - generator = "Ninja" + generator("ninja") variant("python", default=False, description="Install python bindings") variant("tutorials", default=False, description="Install the Halide Tutorials.") variant("utils", default=False, description="Install the Halide Utilities.") @@ -54,7 +54,6 @@ class Halide(CMakePackage, PythonExtension): variant("sharedllvm", default=False, description="Link to the shared version of LLVM.") depends_on("cmake@3.22:", type="build") - depends_on("ninja", type="build") depends_on("llvm+clang+lld build_type=Release", type=("link", "run")) depends_on("llvm@13.0.0:15", type=("link", "run"), when="@14.0.0") depends_on("llvm@14.0.0:16", type=("link", "run"), when="@15.0.0:") diff --git a/var/spack/repos/builtin/packages/hpx/package.py b/var/spack/repos/builtin/packages/hpx/package.py index 43bdcb50d0..5d0ce6698e 100644 --- a/var/spack/repos/builtin/packages/hpx/package.py +++ b/var/spack/repos/builtin/packages/hpx/package.py @@ -36,7 +36,7 @@ class Hpx(CMakePackage, CudaPackage, ROCmPackage): version("1.2.0", sha256="20942314bd90064d9775f63b0e58a8ea146af5260a4c84d0854f9f968077c170") version("1.1.0", sha256="1f28bbe58d8f0da600d60c3a74a644d75ac777b20a018a5c1c6030a470e8a1c9") - generator = "Ninja" + generator("ninja") map_cxxstd = lambda cxxstd: "2a" if cxxstd == "20" else cxxstd cxxstds = ("11", "14", "17", "20") @@ -91,7 +91,6 @@ class Hpx(CMakePackage, CudaPackage, ROCmPackage): # Build dependencies depends_on("python", type=("build", "test", "run")) - depends_on("ninja", type="build") depends_on("pkgconfig", type="build") depends_on("git", type="build") depends_on("cmake", type="build") diff --git a/var/spack/repos/builtin/packages/hydrogen/package.py b/var/spack/repos/builtin/packages/hydrogen/package.py index 17aac5dd8f..df00235725 100644 --- a/var/spack/repos/builtin/packages/hydrogen/package.py +++ b/var/spack/repos/builtin/packages/hydrogen/package.py @@ -139,8 +139,7 @@ class Hydrogen(CMakePackage, CudaPackage, ROCmPackage): msg="Hydrogen did not exist before v0.99. " + "Did you mean to use Elemental instead?", ) - generator = "Ninja" - depends_on("ninja", type="build") + generator("ninja") @property def libs(self): diff --git a/var/spack/repos/builtin/packages/kineto/package.py b/var/spack/repos/builtin/packages/kineto/package.py index c58df1367c..8f0ed34d05 100644 --- a/var/spack/repos/builtin/packages/kineto/package.py +++ b/var/spack/repos/builtin/packages/kineto/package.py @@ -26,9 +26,7 @@ class Kineto(CMakePackage): root_cmakelists_dir = "libkineto" + generator("ninja") depends_on("cmake@3.5:", type="build") - depends_on("ninja", type="build") depends_on("python", type="build") depends_on("cuda") - - generator = "Ninja" diff --git a/var/spack/repos/builtin/packages/lbann/package.py b/var/spack/repos/builtin/packages/lbann/package.py index 366a12ebb2..f351cfaf08 100644 --- a/var/spack/repos/builtin/packages/lbann/package.py +++ b/var/spack/repos/builtin/packages/lbann/package.py @@ -315,8 +315,7 @@ class Lbann(CMakePackage, CudaPackage, ROCmPackage): depends_on("spdlog", when="@:0.90,0.102:") depends_on("zstr") - generator = "Ninja" - depends_on("ninja", type="build") + generator("ninja") @property def common_config_args(self): diff --git a/var/spack/repos/builtin/packages/llvm-amdgpu/package.py b/var/spack/repos/builtin/packages/llvm-amdgpu/package.py index c6157970ae..ed10d33e4a 100644 --- a/var/spack/repos/builtin/packages/llvm-amdgpu/package.py +++ b/var/spack/repos/builtin/packages/llvm-amdgpu/package.py @@ -17,7 +17,7 @@ class LlvmAmdgpu(CMakePackage): git = "https://github.com/RadeonOpenCompute/llvm-project.git" url = "https://github.com/RadeonOpenCompute/llvm-project/archive/rocm-5.4.3.tar.gz" tags = ["rocm"] - generator = "Ninja" + generator("ninja") maintainers("srekolam", "renjithravindrankannath", "haampie") @@ -141,7 +141,6 @@ class LlvmAmdgpu(CMakePackage): depends_on("z3", type="link") depends_on("zlib", type="link") depends_on("ncurses+termlib", type="link") - depends_on("ninja", type="build") depends_on("pkgconfig", type="build") # openmp dependencies diff --git a/var/spack/repos/builtin/packages/llvm-doe/package.py b/var/spack/repos/builtin/packages/llvm-doe/package.py index e9820dfacb..7f312bdb66 100644 --- a/var/spack/repos/builtin/packages/llvm-doe/package.py +++ b/var/spack/repos/builtin/packages/llvm-doe/package.py @@ -26,7 +26,7 @@ class LlvmDoe(CMakePackage, CudaPackage): tags = ["e4s"] - generator = "Ninja" + generator("ninja") family = "compiler" # Used by lmod @@ -117,7 +117,6 @@ class LlvmDoe(CMakePackage, CudaPackage): # Build dependency depends_on("cmake@3.4.3:", type="build") depends_on("cmake@3.13.4:", type="build", when="@12:") - depends_on("ninja", type="build") depends_on("python", when="~python", type="build") depends_on("pkgconfig", type="build") diff --git a/var/spack/repos/builtin/packages/llvm-openmp-ompt/package.py b/var/spack/repos/builtin/packages/llvm-openmp-ompt/package.py index ec56a106da..d4aaf43ce6 100644 --- a/var/spack/repos/builtin/packages/llvm-openmp-ompt/package.py +++ b/var/spack/repos/builtin/packages/llvm-openmp-ompt/package.py @@ -48,7 +48,7 @@ class LlvmOpenmpOmpt(CMakePackage): depends_on("elf", when="+libomptarget") depends_on("libffi", when="+libomptarget") - generator = "Ninja" + generator("ninja") def cmake_args(self): cmake_args = [ diff --git a/var/spack/repos/builtin/packages/llvm/package.py b/var/spack/repos/builtin/packages/llvm/package.py index e94ff94d75..de186b0943 100644 --- a/var/spack/repos/builtin/packages/llvm/package.py +++ b/var/spack/repos/builtin/packages/llvm/package.py @@ -30,7 +30,7 @@ class Llvm(CMakePackage, CudaPackage): tags = ["e4s"] - generator = "Ninja" + generator("ninja") family = "compiler" # Used by lmod @@ -212,7 +212,6 @@ class Llvm(CMakePackage, CudaPackage): # Build dependency depends_on("cmake@3.4.3:", type="build") depends_on("cmake@3.13.4:", type="build", when="@12:") - depends_on("ninja", type="build") depends_on("python", when="~python", type="build") depends_on("pkgconfig", type="build") diff --git a/var/spack/repos/builtin/packages/mxnet/package.py b/var/spack/repos/builtin/packages/mxnet/package.py index 2d68c03b7d..c23b5c381a 100644 --- a/var/spack/repos/builtin/packages/mxnet/package.py +++ b/var/spack/repos/builtin/packages/mxnet/package.py @@ -47,8 +47,8 @@ class Mxnet(CMakePackage, CudaPackage, PythonExtension): variant("mkldnn", default=False, description="Build with MKL-DNN support") variant("python", default=True, description="Install python bindings") + generator("ninja") depends_on("cmake@3.13:", type="build") - depends_on("ninja", type="build") depends_on("pkgconfig", when="@1.6.0", type="build") depends_on("blas") depends_on("cuda@:10.2", when="@:1.8.0 +cuda") @@ -84,7 +84,6 @@ class Mxnet(CMakePackage, CudaPackage, PythonExtension): # python/setup.py assumes libs can be found in build directory build_directory = "build" - generator = "Ninja" def setup_run_environment(self, env): env.set("MXNET_LIBRARY_PATH", self.spec["mxnet"].libs[0]) diff --git a/var/spack/repos/builtin/packages/nnpack/package.py b/var/spack/repos/builtin/packages/nnpack/package.py index cc39ee6dcb..0a63404f7f 100644 --- a/var/spack/repos/builtin/packages/nnpack/package.py +++ b/var/spack/repos/builtin/packages/nnpack/package.py @@ -20,13 +20,11 @@ class Nnpack(CMakePackage): version("2018-05-21", commit="3eb0d453662d05a708f43b108bed9e17b705383e") # py-torch@0.4.1 version("2018-04-05", commit="b63fe1ba8963f1756b8decc593766615cee99c35") # py-torch@:0.4.0 + generator("ninja") depends_on("cmake@2.8.12:", type="build") - depends_on("ninja", type="build") depends_on("python", type="build") depends_on("py-setuptools", type="build") - generator = "Ninja" - resource( name="six", url="https://files.pythonhosted.org/packages/source/s/six/six-1.11.0.tar.gz", diff --git a/var/spack/repos/builtin/packages/onnx/package.py b/var/spack/repos/builtin/packages/onnx/package.py index b8c3eb6a7a..3fad7d5b85 100644 --- a/var/spack/repos/builtin/packages/onnx/package.py +++ b/var/spack/repos/builtin/packages/onnx/package.py @@ -50,13 +50,11 @@ class Onnx(CMakePackage): "1.1.0_2018-04-19", commit="7e1bed51cc508a25b22130de459830b5d5063c41" ) # py-torch@0.4.0 + generator("ninja") depends_on("cmake@3.1:", type="build") - depends_on("ninja", type="build") depends_on("python", type="build") depends_on("protobuf") - generator = "Ninja" - def cmake_args(self): # Try to get ONNX to use the same version of python as the spec is using args = ["-DPY_VERSION={0}".format(self.spec["python"].version.up_to(2))] diff --git a/var/spack/repos/builtin/packages/paraview/package.py b/var/spack/repos/builtin/packages/paraview/package.py index aed59960bc..fca9d542f2 100644 --- a/var/spack/repos/builtin/packages/paraview/package.py +++ b/var/spack/repos/builtin/packages/paraview/package.py @@ -148,8 +148,6 @@ class Paraview(CMakePackage, CudaPackage, ROCmPackage): depends_on("cmake@3.3:", type="build") depends_on("cmake@3.21:", type="build", when="+rocm") - depends_on("ninja", type="build") - extends("python", when="+python") # VTK < 8.2.1 can't handle Python 3.8 @@ -276,13 +274,10 @@ class Paraview(CMakePackage, CudaPackage, ROCmPackage): # Fix VTK to work with external freetype using CONFIG mode for find_package patch("FindFreetype.cmake.patch", when="@5.10.1:") - @property - def generator(self): - # https://gitlab.kitware.com/paraview/paraview/-/issues/21223 - if self.spec.satisfies("%xl") or self.spec.satisfies("%xl_r"): - return "Unix Makefiles" - else: - return "Ninja" + generator("ninja", "make", default="ninja") + # https://gitlab.kitware.com/paraview/paraview/-/issues/21223 + conflicts("generator=ninja", when="%xl") + conflicts("generator=ninja", when="%xl_r") def url_for_version(self, version): _urlfmt = "http://www.paraview.org/files/v{0}/ParaView-v{1}{2}.tar.{3}" diff --git a/var/spack/repos/builtin/packages/pika-algorithms/package.py b/var/spack/repos/builtin/packages/pika-algorithms/package.py index ff99934f17..6e142c0348 100644 --- a/var/spack/repos/builtin/packages/pika-algorithms/package.py +++ b/var/spack/repos/builtin/packages/pika-algorithms/package.py @@ -20,7 +20,7 @@ class PikaAlgorithms(CMakePackage): version("0.1.0", sha256="64da008897dfa7373155595c46d2ce6b97a8a3cb5bea33ae7f2d1ff359f0d9b6") version("main", branch="main") - generator = "Ninja" + generator("ninja") map_cxxstd = lambda cxxstd: "2a" if cxxstd == "20" else cxxstd cxxstds = ("17", "20") @@ -33,7 +33,6 @@ class PikaAlgorithms(CMakePackage): # Build dependencies depends_on("git", type="build") - depends_on("ninja", type="build") depends_on("cmake@3.22:", type="build") conflicts("%gcc@:8") diff --git a/var/spack/repos/builtin/packages/pika/package.py b/var/spack/repos/builtin/packages/pika/package.py index 4e46a7a7db..8c9c01ccd6 100644 --- a/var/spack/repos/builtin/packages/pika/package.py +++ b/var/spack/repos/builtin/packages/pika/package.py @@ -32,7 +32,7 @@ class Pika(CMakePackage, CudaPackage, ROCmPackage): version("0.1.0", sha256="aa0ae2396cd264d821a73c4c7ecb118729bb3de042920c9248909d33755e7327") version("main", branch="main") - generator = "Ninja" + generator("ninja") map_cxxstd = lambda cxxstd: "2a" if cxxstd == "20" else cxxstd cxxstds = ("17", "20") @@ -73,7 +73,6 @@ class Pika(CMakePackage, CudaPackage, ROCmPackage): # Build dependencies depends_on("git", type="build") - depends_on("ninja", type="build") depends_on("cmake@3.18:", type="build") depends_on("cmake@3.22:", when="@0.8:", type="build") diff --git a/var/spack/repos/builtin/packages/psimd/package.py b/var/spack/repos/builtin/packages/psimd/package.py index c79c56c173..4f8c8d8bf1 100644 --- a/var/spack/repos/builtin/packages/psimd/package.py +++ b/var/spack/repos/builtin/packages/psimd/package.py @@ -18,7 +18,5 @@ class Psimd(CMakePackage): version("2018-09-06", commit="90a938f30ba414ada2f4b00674ee9631d7d85e19") # py-torch@1.0:1.4 version("2017-10-26", commit="4ac61b112252778b174575931c641bef661ab3cd") # py-torch@0.4 + generator("ninja") depends_on("cmake@2.8.12:", type="build") - depends_on("ninja", type="build") - - generator = "Ninja" diff --git a/var/spack/repos/builtin/packages/pthreadpool/package.py b/var/spack/repos/builtin/packages/pthreadpool/package.py index 5dc0fa0b21..af19b7ad29 100644 --- a/var/spack/repos/builtin/packages/pthreadpool/package.py +++ b/var/spack/repos/builtin/packages/pthreadpool/package.py @@ -20,12 +20,10 @@ class Pthreadpool(CMakePackage): version("2018-10-08", commit="13da0b4c21d17f94150713366420baaf1b5a46f4") # py-torch@1.0:1.4 version("2018-02-25", commit="2b06b31f6a315162348e1f3c24325eedaf6cc559") # py-torch@:0.4 + generator("ninja") depends_on("cmake@3.5:", type="build") - depends_on("ninja", type="build") depends_on("python", type="build") - generator = "Ninja" - resource( name="fxdiv", git="https://github.com/Maratyszcza/FXdiv.git", diff --git a/var/spack/repos/builtin/packages/py-onnx-runtime/package.py b/var/spack/repos/builtin/packages/py-onnx-runtime/package.py index c3341e889b..349f564d86 100644 --- a/var/spack/repos/builtin/packages/py-onnx-runtime/package.py +++ b/var/spack/repos/builtin/packages/py-onnx-runtime/package.py @@ -28,7 +28,6 @@ class PyOnnxRuntime(CMakePackage, PythonExtension): variant("cuda", default=False, description="Build with CUDA support") depends_on("cmake@3.1:", type="build") - depends_on("ninja", type="build") depends_on("python", type=("build", "run")) depends_on("py-pip", type="build") depends_on("protobuf") @@ -74,7 +73,7 @@ class PyOnnxRuntime(CMakePackage, PythonExtension): description="AVX support level", ) - generator = "Ninja" + generator("ninja") root_cmakelists_dir = "cmake" build_directory = "." diff --git a/var/spack/repos/builtin/packages/py-onnxruntime/package.py b/var/spack/repos/builtin/packages/py-onnxruntime/package.py index 946ff5ee89..7efd37f8ec 100644 --- a/var/spack/repos/builtin/packages/py-onnxruntime/package.py +++ b/var/spack/repos/builtin/packages/py-onnxruntime/package.py @@ -25,7 +25,6 @@ class PyOnnxruntime(CMakePackage, PythonExtension): variant("cuda", default=False, description="Build with CUDA support") depends_on("cmake@3.1:", type="build") - depends_on("ninja", type="build") depends_on("python", type=("build", "run")) depends_on("py-pip", type="build") depends_on("protobuf") @@ -71,7 +70,7 @@ class PyOnnxruntime(CMakePackage, PythonExtension): description="AVX support level", ) - generator = "Ninja" + generator("ninja") root_cmakelists_dir = "cmake" build_directory = "." diff --git a/var/spack/repos/builtin/packages/py-pybind11/package.py b/var/spack/repos/builtin/packages/py-pybind11/package.py index 180a1c1590..d320b46023 100644 --- a/var/spack/repos/builtin/packages/py-pybind11/package.py +++ b/var/spack/repos/builtin/packages/py-pybind11/package.py @@ -59,7 +59,7 @@ class PyPybind11(CMakePackage, PythonExtension): extends("python") with when("build_system=cmake"): - depends_on("ninja", type="build") + generator("ninja") depends_on("cmake@3.13:", type="build") depends_on("cmake@3.18:", type="build", when="@2.6.0:") diff --git a/var/spack/repos/builtin/packages/qnnpack/package.py b/var/spack/repos/builtin/packages/qnnpack/package.py index 005f2fba77..e641959919 100644 --- a/var/spack/repos/builtin/packages/qnnpack/package.py +++ b/var/spack/repos/builtin/packages/qnnpack/package.py @@ -20,8 +20,8 @@ class Qnnpack(CMakePackage): version("2018-12-27", commit="6c62fddc6d15602be27e9e4cbb9e985151d2fa82") # py-torch@1.2 version("2018-12-04", commit="ef05e87cef6b8e719989ce875b5e1c9fdb304c05") # py-torch@1.0:1.1 + generator("ninja") depends_on("cmake@3.5:", type="build") - depends_on("ninja", type="build") depends_on("python", type="build") resource( @@ -69,8 +69,6 @@ class Qnnpack(CMakePackage): placement="pthreadpool", ) - generator = "Ninja" - def cmake_args(self): return [ self.define( diff --git a/var/spack/repos/builtin/packages/qt-base/package.py b/var/spack/repos/builtin/packages/qt-base/package.py index 7170de50dc..e8ff25bbc6 100644 --- a/var/spack/repos/builtin/packages/qt-base/package.py +++ b/var/spack/repos/builtin/packages/qt-base/package.py @@ -31,13 +31,11 @@ class QtPackage(CMakePackage): _list_url = "https://github.com/qt/{}/tags" return _list_url.format(qualname.lower()) - generator = "Ninja" - maintainers("wdconinc", "sethrj") # Default dependencies for all qt-* components + generator("ninja") depends_on("cmake@3.16:", type="build") - depends_on("ninja", type="build") depends_on("pkgconfig", type="build") depends_on("python", type="build") diff --git a/var/spack/repos/builtin/packages/sleef/package.py b/var/spack/repos/builtin/packages/sleef/package.py index 1040c64b98..05061d6ec0 100644 --- a/var/spack/repos/builtin/packages/sleef/package.py +++ b/var/spack/repos/builtin/packages/sleef/package.py @@ -51,10 +51,8 @@ class Sleef(CMakePackage): values=("Debug", "Release", "RelWithDebInfo", "MinSizeRel"), ) + generator("ninja") depends_on("cmake@3.4.3:", type="build") - depends_on("ninja", type="build") - - generator = "Ninja" def cmake_args(self): return [ diff --git a/var/spack/repos/builtin/packages/spectre/package.py b/var/spack/repos/builtin/packages/spectre/package.py index 62d1ad3336..a46e134162 100644 --- a/var/spack/repos/builtin/packages/spectre/package.py +++ b/var/spack/repos/builtin/packages/spectre/package.py @@ -26,7 +26,7 @@ class Spectre(CMakePackage): maintainers("nilsvu") - generator = "Ninja" + generator("ninja") version("develop", branch="develop") version( @@ -146,7 +146,6 @@ class Spectre(CMakePackage): # Build dependencies depends_on("cmake@3.12:", type="build") - depends_on("ninja", type="build") depends_on("python@2.7:", type="build") # Link dependencies diff --git a/var/spack/repos/builtin/packages/tensorpipe/package.py b/var/spack/repos/builtin/packages/tensorpipe/package.py index 52ee112bfd..3db5a5a240 100644 --- a/var/spack/repos/builtin/packages/tensorpipe/package.py +++ b/var/spack/repos/builtin/packages/tensorpipe/package.py @@ -29,8 +29,6 @@ class Tensorpipe(CMakePackage): "2020-06-26", commit="3b8089c9c6717038cff44b70b881d0ad6c93e679", submodules=True ) # py-torch@1.6 + generator("ninja") depends_on("cmake@3.5:", type="build") - depends_on("ninja", type="build") depends_on("libuv@1.26:") - - generator = "Ninja" diff --git a/var/spack/repos/builtin/packages/tiramisu/package.py b/var/spack/repos/builtin/packages/tiramisu/package.py index 991f1f0aaf..055cd4c0a2 100644 --- a/var/spack/repos/builtin/packages/tiramisu/package.py +++ b/var/spack/repos/builtin/packages/tiramisu/package.py @@ -16,7 +16,7 @@ class Tiramisu(CMakePackage, CudaPackage, PythonExtension): maintainers("wraith1995") - generator = "Ninja" + generator("ninja") version("master", branch="master") version("2023-2-8", commit="2cd0c43cc1656bfa43cfb6e81d06f770cbf7251e") @@ -32,8 +32,6 @@ class Tiramisu(CMakePackage, CudaPackage, PythonExtension): ) depends_on("cmake@3.5:", type="build") - depends_on("ninja", type="build") - depends_on("halide@14.0.0:", type=("build", "link", "run")) depends_on("isl", type=("build", "link", "run")) depends_on("python@3.8:", type=("build", "link", "run"), when="+python") diff --git a/var/spack/repos/builtin/packages/visit/package.py b/var/spack/repos/builtin/packages/visit/package.py index 55940c96b9..167957c93c 100644 --- a/var/spack/repos/builtin/packages/visit/package.py +++ b/var/spack/repos/builtin/packages/visit/package.py @@ -69,7 +69,7 @@ class Visit(CMakePackage): version("3.0.1", sha256="a506d4d83b8973829e68787d8d721199523ce7ec73e7594e93333c214c2c12bd") root_cmakelists_dir = "src" - generator = "Ninja" + generator("ninja") variant("gui", default=True, description="Enable VisIt's GUI") variant("osmesa", default=False, description="Use OSMesa for off-screen CPU rendering") @@ -99,7 +99,6 @@ class Visit(CMakePackage): conflicts("+gui", when="+osmesa") depends_on("cmake@3.14.7:", type="build") - depends_on("ninja", type="build") depends_on("mpi", when="+mpi") diff --git a/var/spack/repos/builtin/packages/xgboost/package.py b/var/spack/repos/builtin/packages/xgboost/package.py index bc8104378a..5996799e9e 100644 --- a/var/spack/repos/builtin/packages/xgboost/package.py +++ b/var/spack/repos/builtin/packages/xgboost/package.py @@ -32,7 +32,6 @@ class Xgboost(CMakePackage, CudaPackage): depends_on("cmake@3.13:", type="build") depends_on("cmake@3.16:", when="platform=darwin", type="build") - depends_on("ninja", type="build") depends_on("cuda@10:", when="+cuda") # https://github.com/dmlc/xgboost/pull/7379 depends_on("cuda@10:11.4", when="@:1.5.0+cuda") @@ -52,7 +51,7 @@ class Xgboost(CMakePackage, CudaPackage): "https://developer.nvidia.com/cuda-gpus", ) - generator = "Ninja" + generator("ninja") def cmake_args(self): # https://xgboost.readthedocs.io/en/latest/build.html diff --git a/var/spack/repos/builtin/packages/xnnpack/package.py b/var/spack/repos/builtin/packages/xnnpack/package.py index 08f8d65c22..4307c2da2b 100644 --- a/var/spack/repos/builtin/packages/xnnpack/package.py +++ b/var/spack/repos/builtin/packages/xnnpack/package.py @@ -20,12 +20,10 @@ class Xnnpack(CMakePackage): version("2020-03-23", commit="1b354636b5942826547055252f3b359b54acff95") # py-torch@1.6:1.7 version("2020-02-24", commit="7493bfb9d412e59529bcbced6a902d44cfa8ea1c") # py-torch@1.5 + generator("ninja") depends_on("cmake@3.5:", type="build") - depends_on("ninja", type="build") depends_on("python", type="build") - generator = "Ninja" - resource( name="clog", url="https://github.com/pytorch/cpuinfo/archive/d5e37adf1406cf899d7d9ec1d317c47506ccb970.tar.gz", -- cgit v1.2.3-60-g2f50