summaryrefslogtreecommitdiff
path: root/var
diff options
context:
space:
mode:
authorWouter Deconinck <wdconinc@gmail.com>2023-10-01 07:33:37 -0500
committerGitHub <noreply@github.com>2023-10-01 21:33:37 +0900
commitb25f8643ff6f28d9ca0c23d4eb46aadb840683cf (patch)
tree11da7474d395472beb635ac5612dfd117850440f /var
parent9120b6644d5b5deb391a7d96da581ac86f1dd8a1 (diff)
downloadspack-b25f8643ff6f28d9ca0c23d4eb46aadb840683cf.tar.gz
spack-b25f8643ff6f28d9ca0c23d4eb46aadb840683cf.tar.bz2
spack-b25f8643ff6f28d9ca0c23d4eb46aadb840683cf.tar.xz
spack-b25f8643ff6f28d9ca0c23d4eb46aadb840683cf.zip
geant4, vecgeom: support variant cxxstd=20 (#39785)
Diffstat (limited to 'var')
-rw-r--r--var/spack/repos/builtin/packages/geant4/package.py58
-rw-r--r--var/spack/repos/builtin/packages/vecgeom/package.py21
2 files changed, 49 insertions, 30 deletions
diff --git a/var/spack/repos/builtin/packages/geant4/package.py b/var/spack/repos/builtin/packages/geant4/package.py
index 47a521955b..bf4ade6ce7 100644
--- a/var/spack/repos/builtin/packages/geant4/package.py
+++ b/var/spack/repos/builtin/packages/geant4/package.py
@@ -4,6 +4,7 @@
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
from spack.package import *
+from spack.variant import _ConditionalVariantValues
class Geant4(CMakePackage):
@@ -43,16 +44,18 @@ class Geant4(CMakePackage):
version("10.4.0", sha256="e919b9b0a88476e00c0b18ab65d40e6a714b55ee4778f66bac32a5396c22aa74")
version("10.3.3", sha256="bcd36a453da44de9368d1d61b0144031a58e4b43a6d2d875e19085f2700a89d8")
- _cxxstd_values = ("11", "14", "17")
+ _cxxstd_values = (
+ conditional("11", "14", when="@:10"),
+ conditional("17", when="@10.4.1:"),
+ conditional("20", when="@10.7.0:"),
+ )
variant(
"cxxstd",
- default=_cxxstd_values[0],
+ default="11",
values=_cxxstd_values,
multi=False,
description="Use the specified C++ standard when building.",
)
- conflicts("cxxstd=11", when="@11:", msg="geant4@11: only supports cxxstd=17")
- conflicts("cxxstd=14", when="@11:", msg="geant4@11: only supports cxxstd=17")
variant("threads", default=True, description="Build with multithreading")
variant("vecgeom", default=False, description="Enable vecgeom support")
@@ -97,30 +100,39 @@ class Geant4(CMakePackage):
depends_on("python@3:", when="+python")
extends("python", when="+python")
- for std in _cxxstd_values:
- # CLHEP version requirements to be reviewed
- depends_on("clhep@2.4.6.0: cxxstd=" + std, when="@11.1: cxxstd=" + std)
-
- depends_on("clhep@2.4.5.1: cxxstd=" + std, when="@11.0.0: cxxstd=" + std)
-
- depends_on("clhep@2.4.4.0: cxxstd=" + std, when="@10.7.0: cxxstd=" + std)
+ # CLHEP version requirements to be reviewed
+ depends_on("clhep@2.4.6.0:", when="@11.1:")
+ depends_on("clhep@2.4.5.1:", when="@11.0.0:")
+ depends_on("clhep@2.4.4.0:", when="@10.7.0:")
+ depends_on("clhep@2.3.3.0:", when="@10.3.3:10.6")
+
+ # Vecgeom specific versions for each Geant4 version
+ with when("+vecgeom"):
+ depends_on("vecgeom@1.2.0:", when="@11.1:")
+ depends_on("vecgeom@1.1.18:1.1", when="@11.0.0:11.0")
+ depends_on("vecgeom@1.1.8:1.1", when="@10.7.0:10.7")
+ depends_on("vecgeom@1.1.5", when="@10.6.0:10.6")
+ depends_on("vecgeom@1.1.0", when="@10.5.0:10.5")
+ depends_on("vecgeom@0.5.2", when="@10.4.0:10.4")
+ depends_on("vecgeom@0.3rc", when="@10.3.0:10.3")
+
+ def std_when(values):
+ for v in values:
+ if isinstance(v, _ConditionalVariantValues):
+ for c in v:
+ yield (c.value, c.when)
+ else:
+ yield (v, "")
- depends_on("clhep@2.3.3.0: cxxstd=" + std, when="@10.3.3:10.6 cxxstd=" + std)
+ for _std, _when in std_when(_cxxstd_values):
+ depends_on(f"clhep cxxstd={_std}", when=f"{_when} cxxstd={_std}")
+ depends_on(f"vecgeom cxxstd={_std}", when=f"{_when} +vecgeom cxxstd={_std}")
# Spack only supports Xerces-c 3 and above, so no version req
- depends_on("xerces-c netaccessor=curl cxxstd=" + std, when="cxxstd=" + std)
-
- # Vecgeom specific versions for each Geant4 version
- depends_on("vecgeom@1.2.0: cxxstd=" + std, when="@11.1: +vecgeom cxxstd=" + std)
- depends_on("vecgeom@1.1.18:1.1 cxxstd=" + std, when="@11.0.0:11.0 +vecgeom cxxstd=" + std)
- depends_on("vecgeom@1.1.8:1.1 cxxstd=" + std, when="@10.7.0:10.7 +vecgeom cxxstd=" + std)
- depends_on("vecgeom@1.1.5 cxxstd=" + std, when="@10.6.0:10.6 +vecgeom cxxstd=" + std)
- depends_on("vecgeom@1.1.0 cxxstd=" + std, when="@10.5.0:10.5 +vecgeom cxxstd=" + std)
- depends_on("vecgeom@0.5.2 cxxstd=" + std, when="@10.4.0:10.4 +vecgeom cxxstd=" + std)
- depends_on("vecgeom@0.3rc cxxstd=" + std, when="@10.3.0:10.3 +vecgeom cxxstd=" + std)
+ depends_on(f"xerces-c netaccessor=curl cxxstd={_std}", when=f"{_when} cxxstd={_std}")
# Boost.python, conflict handled earlier
- depends_on("boost@1.70: +python cxxstd=" + std, when="+python cxxstd=" + std)
+ depends_on(f"boost@1.70: +python cxxstd={_std}", when=f"{_when} +python cxxstd={_std}")
# Visualization driver dependencies
depends_on("gl", when="+opengl")
diff --git a/var/spack/repos/builtin/packages/vecgeom/package.py b/var/spack/repos/builtin/packages/vecgeom/package.py
index 6e8b9be3ad..7a403bdd9e 100644
--- a/var/spack/repos/builtin/packages/vecgeom/package.py
+++ b/var/spack/repos/builtin/packages/vecgeom/package.py
@@ -5,6 +5,7 @@
from spack.package import *
+from spack.variant import _ConditionalVariantValues
class Vecgeom(CMakePackage, CudaPackage):
@@ -138,7 +139,7 @@ class Vecgeom(CMakePackage, CudaPackage):
deprecated=True,
)
- _cxxstd_values = ("11", "14", "17")
+ _cxxstd_values = (conditional("11", "14", when="@:1.1"), "17", conditional("20", when="@1.2:"))
variant(
"cxxstd",
default="17",
@@ -158,8 +159,6 @@ class Vecgeom(CMakePackage, CudaPackage):
depends_on("veccore@0.4.2", when="@:1.0")
conflicts("+cuda", when="@:1.1.5")
- conflicts("cxxstd=14", when="@1.2:")
- conflicts("cxxstd=11", when="@1.2:")
# Fix missing CMAKE_CUDA_STANDARD
patch(
@@ -174,10 +173,18 @@ class Vecgeom(CMakePackage, CudaPackage):
when="@1.1.18 +cuda ^cuda@:11.4",
)
- for std in _cxxstd_values:
- depends_on("geant4 cxxstd=" + std, when="+geant4 cxxstd=" + std)
- depends_on("root cxxstd=" + std, when="+root cxxstd=" + std)
- depends_on("xerces-c cxxstd=" + std, when="+gdml cxxstd=" + std)
+ def std_when(values):
+ for v in values:
+ if isinstance(v, _ConditionalVariantValues):
+ for c in v:
+ yield (c.value, c.when)
+ else:
+ yield (v, "")
+
+ for _std, _when in std_when(_cxxstd_values):
+ depends_on(f"geant4 cxxstd={_std}", when=f"{_when} +geant4 cxxstd={_std}")
+ depends_on(f"root cxxstd={_std}", when=f"{_when} +root cxxstd={_std}")
+ depends_on(f"xerces-c cxxstd={_std}", when=f"{_when} +gdml cxxstd={_std}")
def cmake_args(self):
spec = self.spec