From 3b21ff109f51655f8d7283f3f93d374b3cdf5e45 Mon Sep 17 00:00:00 2001 From: Paul Gessinger Date: Tue, 7 Jan 2025 05:14:50 +0100 Subject: [PATCH] edm4hep: Add lower clang version bounds (#48419) * edm4hep: Add lower clang version bounds EDM4hep 0.99.1 introduced usage of consteval. While this is technically supported in clang versions below 17, the implementation seems to be incomplete and fail compilation of EDM4hep 0.99.1 and up. * Update var/spack/repos/builtin/packages/edm4hep/package.py Co-authored-by: Thomas Madlener --------- Co-authored-by: Thomas Madlener --- var/spack/repos/builtin/packages/edm4hep/package.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/var/spack/repos/builtin/packages/edm4hep/package.py b/var/spack/repos/builtin/packages/edm4hep/package.py index a2af33d3fcc..85c1acad8ef 100644 --- a/var/spack/repos/builtin/packages/edm4hep/package.py +++ b/var/spack/repos/builtin/packages/edm4hep/package.py @@ -90,6 +90,8 @@ class Edm4hep(CMakePackage): # Corresponding changes in EDM4hep landed with https://github.com/key4hep/EDM4hep/pull/314 extends("python", when="@0.10.6:") + conflicts("%clang@:16", when="@0.99.1:", msg="Incomplete consteval support in clang") + def cmake_args(self): args = [ self.define_from_variant("CMAKE_CXX_STANDARD", "cxxstd"),