From 18152b9b0ff91ff8806fa2761c2e29d2db13067d Mon Sep 17 00:00:00 2001
From: Massimiliano Culpo <massimiliano.culpo@gmail.com>
Date: Wed, 25 Sep 2024 15:01:19 +0200
Subject: [PATCH] builtin.mock et al. : changes to packages

Signed-off-by: Massimiliano Culpo <massimiliano.culpo@gmail.com>
---
 .../builtin.mock/packages/bowtie/package.py   |  5 +-
 .../builtin.mock/packages/callpath/package.py |  2 +
 .../packages/cmake-client/package.py          |  2 +
 .../builtin.mock/packages/cmake/package.py    |  3 +
 .../packages/conflict-parent/package.py       |  1 +
 .../builtin.mock/packages/conflict/package.py |  2 +
 .../packages/dt-diamond-bottom/package.py     |  2 +
 .../packages/dt-diamond-left/package.py       |  1 +
 .../packages/dt-diamond-right/package.py      |  1 +
 .../packages/dt-diamond/package.py            |  2 +
 .../builtin.mock/packages/dyninst/package.py  |  2 +
 .../builtin.mock/packages/fftw/package.py     |  2 +
 .../repos/builtin.mock/packages/gcc-runtime   |  1 +
 .../builtin.mock/packages/gcc/package.py      | 96 ++++++++++++++++++-
 .../intel-oneapi-compilers/package.py         |  2 +
 .../builtin.mock/packages/libdwarf/package.py |  3 +
 .../builtin.mock/packages/libelf/package.py   |  2 +
 .../builtin.mock/packages/llvm/package.py     | 35 +++++++
 .../builtin.mock/packages/mpich/package.py    |  4 +
 .../builtin.mock/packages/mpich2/package.py   |  2 +
 .../builtin.mock/packages/mpileaks/package.py |  2 +
 .../packages/multimethod/package.py           |  3 +-
 .../packages/multivalue-variant/package.py    |  2 +
 .../packages/openblas-with-lapack/package.py  |  1 +
 .../builtin.mock/packages/openblas/package.py |  2 +
 .../builtin.mock/packages/pkg-a/package.py    |  2 +
 .../builtin.mock/packages/pkg-b/package.py    |  1 +
 .../builtin.mock/packages/pkg-c/package.py    | 12 +++
 .../packages/requires_clang/package.py        |  2 +
 .../packages/requires_clang_or_gcc/package.py |  2 +
 .../packages/simple-inheritance/package.py    |  2 +
 .../sticky-variant-dependent/package.py       |  2 +
 .../packages/sticky-variant/package.py        |  2 +
 .../packages/with-constraint-met/package.py   |  2 +
 .../builtin.mock/packages/zmpi/package.py     |  2 +
 .../repos/compiler_runtime.test/packages/gcc  |  1 +
 .../packages/gcc-runtime                      |  1 +
 .../packages/gcc-runtime/package.py           | 13 ---
 .../packages/gcc/package.py                   | 35 -------
 .../compiler_runtime.test/packages/glibc      |  1 +
 .../compiler_runtime.test/packages/gmake      |  1 +
 .../compiler_runtime.test/packages/gnuconfig  |  1 +
 .../packages/pkg-a/package.py                 |  3 +
 .../packages/pkg-b/package.py                 |  3 +
 var/spack/repos/duplicates.test/packages/gcc  |  1 +
 .../duplicates.test/packages/gcc-runtime      |  1 +
 .../repos/duplicates.test/packages/glibc      |  1 +
 .../repos/duplicates.test/packages/gnuconfig  |  1 +
 var/spack/repos/duplicates.test/packages/llvm |  1 +
 var/spack/repos/edges.test/packages/gcc       |  1 +
 .../repos/edges.test/packages/gcc-runtime     |  1 +
 var/spack/repos/edges.test/packages/glibc     |  1 +
 var/spack/repos/edges.test/packages/gmake     |  1 +
 var/spack/repos/edges.test/packages/gnuconfig |  1 +
 var/spack/repos/edges.test/packages/llvm      |  1 +
 var/spack/repos/flags.test/packages/gcc       |  1 +
 .../repos/flags.test/packages/gcc-runtime     |  1 +
 var/spack/repos/flags.test/packages/glibc     |  1 +
 var/spack/repos/flags.test/packages/gmake     |  1 +
 var/spack/repos/flags.test/packages/gnuconfig |  1 +
 .../repos/requirements.test/packages/gcc      |  1 +
 .../requirements.test/packages/gcc-runtime    |  1 +
 .../repos/requirements.test/packages/glibc    |  1 +
 .../repos/requirements.test/packages/gmake    |  1 +
 .../requirements.test/packages/gnuconfig      |  1 +
 .../repos/requirements.test/packages/llvm     |  1 +
 66 files changed, 239 insertions(+), 51 deletions(-)
 create mode 120000 var/spack/repos/builtin.mock/packages/gcc-runtime
 create mode 120000 var/spack/repos/compiler_runtime.test/packages/gcc
 create mode 120000 var/spack/repos/compiler_runtime.test/packages/gcc-runtime
 delete mode 100644 var/spack/repos/compiler_runtime.test/packages/gcc-runtime/package.py
 delete mode 100644 var/spack/repos/compiler_runtime.test/packages/gcc/package.py
 create mode 120000 var/spack/repos/compiler_runtime.test/packages/glibc
 create mode 120000 var/spack/repos/compiler_runtime.test/packages/gmake
 create mode 120000 var/spack/repos/compiler_runtime.test/packages/gnuconfig
 create mode 120000 var/spack/repos/duplicates.test/packages/gcc
 create mode 120000 var/spack/repos/duplicates.test/packages/gcc-runtime
 create mode 120000 var/spack/repos/duplicates.test/packages/glibc
 create mode 120000 var/spack/repos/duplicates.test/packages/gnuconfig
 create mode 120000 var/spack/repos/duplicates.test/packages/llvm
 create mode 120000 var/spack/repos/edges.test/packages/gcc
 create mode 120000 var/spack/repos/edges.test/packages/gcc-runtime
 create mode 120000 var/spack/repos/edges.test/packages/glibc
 create mode 120000 var/spack/repos/edges.test/packages/gmake
 create mode 120000 var/spack/repos/edges.test/packages/gnuconfig
 create mode 120000 var/spack/repos/edges.test/packages/llvm
 create mode 120000 var/spack/repos/flags.test/packages/gcc
 create mode 120000 var/spack/repos/flags.test/packages/gcc-runtime
 create mode 120000 var/spack/repos/flags.test/packages/glibc
 create mode 120000 var/spack/repos/flags.test/packages/gmake
 create mode 120000 var/spack/repos/flags.test/packages/gnuconfig
 create mode 120000 var/spack/repos/requirements.test/packages/gcc
 create mode 120000 var/spack/repos/requirements.test/packages/gcc-runtime
 create mode 120000 var/spack/repos/requirements.test/packages/glibc
 create mode 120000 var/spack/repos/requirements.test/packages/gmake
 create mode 120000 var/spack/repos/requirements.test/packages/gnuconfig
 create mode 120000 var/spack/repos/requirements.test/packages/llvm

diff --git a/var/spack/repos/builtin.mock/packages/bowtie/package.py b/var/spack/repos/builtin.mock/packages/bowtie/package.py
index ba5c60223bf..42497fd6d66 100644
--- a/var/spack/repos/builtin.mock/packages/bowtie/package.py
+++ b/var/spack/repos/builtin.mock/packages/bowtie/package.py
@@ -16,5 +16,8 @@ class Bowtie(Package):
     version("1.2.2", md5="1c837ecd990bb022d07e7aab32b09847")
     version("1.2.0", md5="1c837ecd990bb022d07e7aab32b09847")
 
+    depends_on("c", type="build")
+
     conflicts("%gcc@:4.5.0", when="@1.2.2")
-    conflicts("%gcc@:10.2.1", when="@:1.3.0")
+    conflicts("%gcc@:10.2.1", when="@:1.2.9")
+    conflicts("%gcc", when="@1.3")
diff --git a/var/spack/repos/builtin.mock/packages/callpath/package.py b/var/spack/repos/builtin.mock/packages/callpath/package.py
index a960f9866a0..347fe8d9a6a 100644
--- a/var/spack/repos/builtin.mock/packages/callpath/package.py
+++ b/var/spack/repos/builtin.mock/packages/callpath/package.py
@@ -14,6 +14,8 @@ class Callpath(Package):
     version("0.9", md5="0123456789abcdef0123456789abcdef")
     version("1.0", md5="0123456789abcdef0123456789abcdef")
 
+    depends_on("c", type="build")
+
     depends_on("dyninst")
     depends_on("mpi")
 
diff --git a/var/spack/repos/builtin.mock/packages/cmake-client/package.py b/var/spack/repos/builtin.mock/packages/cmake-client/package.py
index b196e089911..88080393feb 100644
--- a/var/spack/repos/builtin.mock/packages/cmake-client/package.py
+++ b/var/spack/repos/builtin.mock/packages/cmake-client/package.py
@@ -30,6 +30,8 @@ class CmakeClient(CMakePackage):
     variant("single", description="", default="blue", values=("blue", "red", "green"), multi=False)
     variant("truthy", description="", default=True)
 
+    depends_on("c", type="build")
+
     callback_counter = 0
 
     flipped = False
diff --git a/var/spack/repos/builtin.mock/packages/cmake/package.py b/var/spack/repos/builtin.mock/packages/cmake/package.py
index 670c170eebb..b5f962b40c8 100644
--- a/var/spack/repos/builtin.mock/packages/cmake/package.py
+++ b/var/spack/repos/builtin.mock/packages/cmake/package.py
@@ -21,6 +21,9 @@ class Cmake(Package):
     homepage = "https://www.cmake.org"
     url = "https://cmake.org/files/v3.4/cmake-3.4.3.tar.gz"
 
+    depends_on("c", type="build")
+    depends_on("cxx", type="build")
+
     version(
         "3.23.1",
         md5="4cb3ff35b2472aae70f542116d616e63",
diff --git a/var/spack/repos/builtin.mock/packages/conflict-parent/package.py b/var/spack/repos/builtin.mock/packages/conflict-parent/package.py
index 65179dd59f9..bc2f04757aa 100644
--- a/var/spack/repos/builtin.mock/packages/conflict-parent/package.py
+++ b/var/spack/repos/builtin.mock/packages/conflict-parent/package.py
@@ -15,6 +15,7 @@ class ConflictParent(Package):
     version("1.0", md5="0123456789abcdef0123456789abcdef")
 
     depends_on("conflict")
+    depends_on("c", type="build")
 
     conflicts("^conflict~foo", when="@0.9")
 
diff --git a/var/spack/repos/builtin.mock/packages/conflict/package.py b/var/spack/repos/builtin.mock/packages/conflict/package.py
index a2b6bacae9a..152c8384f3e 100644
--- a/var/spack/repos/builtin.mock/packages/conflict/package.py
+++ b/var/spack/repos/builtin.mock/packages/conflict/package.py
@@ -18,6 +18,8 @@ class Conflict(Package):
 
     conflicts("%clang", when="+foo")
 
+    depends_on("c", type="build")
+
     def install(self, spec, prefix):
         configure("--prefix=%s" % prefix)
         make()
diff --git a/var/spack/repos/builtin.mock/packages/dt-diamond-bottom/package.py b/var/spack/repos/builtin.mock/packages/dt-diamond-bottom/package.py
index 43ee6ee605d..d540a80a440 100644
--- a/var/spack/repos/builtin.mock/packages/dt-diamond-bottom/package.py
+++ b/var/spack/repos/builtin.mock/packages/dt-diamond-bottom/package.py
@@ -13,3 +13,5 @@ class DtDiamondBottom(Package):
     url = "http://www.example.com/dt-diamond-bottom-1.0.tar.gz"
 
     version("1.0", md5="0123456789abcdef0123456789abcdef")
+
+    depends_on("c", type="build")
diff --git a/var/spack/repos/builtin.mock/packages/dt-diamond-left/package.py b/var/spack/repos/builtin.mock/packages/dt-diamond-left/package.py
index 5ec1d3d880d..fc5e021376f 100644
--- a/var/spack/repos/builtin.mock/packages/dt-diamond-left/package.py
+++ b/var/spack/repos/builtin.mock/packages/dt-diamond-left/package.py
@@ -15,3 +15,4 @@ class DtDiamondLeft(Package):
     version("1.0", md5="0123456789abcdef0123456789abcdef")
 
     depends_on("dt-diamond-bottom", type="build")
+    depends_on("c", type="build")
diff --git a/var/spack/repos/builtin.mock/packages/dt-diamond-right/package.py b/var/spack/repos/builtin.mock/packages/dt-diamond-right/package.py
index ed07cb222fa..812c605a7c4 100644
--- a/var/spack/repos/builtin.mock/packages/dt-diamond-right/package.py
+++ b/var/spack/repos/builtin.mock/packages/dt-diamond-right/package.py
@@ -15,3 +15,4 @@ class DtDiamondRight(Package):
     version("1.0", md5="0123456789abcdef0123456789abcdef")
 
     depends_on("dt-diamond-bottom", type=("build", "link", "run"))
+    depends_on("c", type="build")
diff --git a/var/spack/repos/builtin.mock/packages/dt-diamond/package.py b/var/spack/repos/builtin.mock/packages/dt-diamond/package.py
index 580b06682f6..4b358d9466f 100644
--- a/var/spack/repos/builtin.mock/packages/dt-diamond/package.py
+++ b/var/spack/repos/builtin.mock/packages/dt-diamond/package.py
@@ -16,3 +16,5 @@ class DtDiamond(Package):
 
     depends_on("dt-diamond-left")
     depends_on("dt-diamond-right")
+
+    depends_on("c", type="build")
diff --git a/var/spack/repos/builtin.mock/packages/dyninst/package.py b/var/spack/repos/builtin.mock/packages/dyninst/package.py
index a7a45f2ab10..ad495f3b37c 100644
--- a/var/spack/repos/builtin.mock/packages/dyninst/package.py
+++ b/var/spack/repos/builtin.mock/packages/dyninst/package.py
@@ -29,6 +29,8 @@ class Dyninst(Package):
     depends_on("libelf")
     depends_on("libdwarf")
 
+    depends_on("c", type="build")
+
     def install(self, spec, prefix):
         mkdirp(prefix)
         touch(join_path(prefix, "dummyfile"))
diff --git a/var/spack/repos/builtin.mock/packages/fftw/package.py b/var/spack/repos/builtin.mock/packages/fftw/package.py
index e4d63aa93ea..c93df568435 100644
--- a/var/spack/repos/builtin.mock/packages/fftw/package.py
+++ b/var/spack/repos/builtin.mock/packages/fftw/package.py
@@ -19,4 +19,6 @@ class Fftw(Package):
 
     variant("mpi", default=False, description="Enable MPI")
 
+    depends_on("c", type="build")
+
     depends_on("mpi", when="+mpi")
diff --git a/var/spack/repos/builtin.mock/packages/gcc-runtime b/var/spack/repos/builtin.mock/packages/gcc-runtime
new file mode 120000
index 00000000000..6f620d9d8c2
--- /dev/null
+++ b/var/spack/repos/builtin.mock/packages/gcc-runtime
@@ -0,0 +1 @@
+../../builtin/packages/gcc-runtime
\ No newline at end of file
diff --git a/var/spack/repos/builtin.mock/packages/gcc/package.py b/var/spack/repos/builtin.mock/packages/gcc/package.py
index fa2b0309ff1..eea9b364a9c 100644
--- a/var/spack/repos/builtin.mock/packages/gcc/package.py
+++ b/var/spack/repos/builtin.mock/packages/gcc/package.py
@@ -2,6 +2,7 @@
 # Spack Project Developers. See the top-level COPYRIGHT file for details.
 #
 # SPDX-License-Identifier: (Apache-2.0 OR MIT)
+import os.path
 
 from spack.package import *
 
@@ -24,7 +25,9 @@ class Gcc(CompilerPackage, Package):
         description="Compilers and runtime libraries to build",
     )
 
-    depends_on("conflict", when="@3.0")
+    provides("c", when="languages=c")
+    provides("cxx", when="languages=c++")
+    provides("fortran", when="languages=fortran")
 
     c_names = ["gcc"]
     cxx_names = ["g++"]
@@ -34,9 +37,100 @@ class Gcc(CompilerPackage, Package):
     compiler_version_regex = r"(?<!clang version)\s?([0-9.]+)"
     compiler_version_argument = ("-dumpfullversion", "-dumpversion")
 
+    link_paths = {
+        "c": os.path.join("gcc", "gcc"),
+        "cxx": os.path.join("gcc", "g++"),
+        "fortran": os.path.join("gcc", "gfortran"),
+    }
+
     def install(self, spec, prefix):
         # Create the minimal compiler that will fool `spack compiler find`
         mkdirp(prefix.bin)
         with open(prefix.bin.gcc, "w") as f:
             f.write('#!/bin/bash\necho "%s"' % str(spec.version))
         set_executable(prefix.bin.gcc)
+
+    @property
+    def cc(self):
+        msg = "cannot retrieve C compiler [spec is not concrete]"
+        assert self.spec.concrete, msg
+        if self.spec.external:
+            return self.spec.extra_attributes["compilers"].get("c", None)
+        result = None
+        if "languages=c" in self.spec:
+            result = str(self.spec.prefix.bin.gcc)
+        return result
+
+    @property
+    def cxx(self):
+        msg = "cannot retrieve C++ compiler [spec is not concrete]"
+        assert self.spec.concrete, msg
+        if self.spec.external:
+            return self.spec.extra_attributes["compilers"].get("cxx", None)
+        result = None
+        if "languages=c++" in self.spec:
+            result = os.path.join(self.spec.prefix.bin, "g++")
+        return result
+
+    @property
+    def fortran(self):
+        msg = "cannot retrieve Fortran compiler [spec is not concrete]"
+        assert self.spec.concrete, msg
+        if self.spec.external:
+            return self.spec.extra_attributes["compilers"].get("fortran", None)
+        result = None
+        if "languages=fortran" in self.spec:
+            result = str(self.spec.prefix.bin.gfortran)
+        return result
+
+    @classmethod
+    def runtime_constraints(cls, *, spec, pkg):
+        """Callback function to inject runtime-related rules into the solver.
+
+        Rule-injection is obtained through method calls of the ``pkg`` argument.
+
+        Documentation for this function is temporary. When the API will be in its final state,
+        we'll document the behavior at https://spack.readthedocs.io/en/latest/
+
+        Args:
+            spec: spec that will inject runtime dependencies
+            pkg: object used to forward information to the solver
+        """
+        pkg("*").depends_on(
+            "gcc-runtime",
+            when="%gcc",
+            type="link",
+            description="If any package uses %gcc, it depends on gcc-runtime",
+        )
+        pkg("*").depends_on(
+            f"gcc-runtime@{str(spec.version)}:",
+            when=f"^[deptypes=build] {spec.name}@{spec.versions}",
+            type="link",
+            description=f"If any package uses %{str(spec)}, "
+            f"it depends on gcc-runtime@{str(spec.version)}:",
+        )
+
+        gfortran_str = "libgfortran@5"
+        if spec.satisfies("gcc@:6"):
+            gfortran_str = "libgfortran@3"
+        elif spec.satisfies("gcc@7"):
+            gfortran_str = "libgfortran@4"
+
+        for fortran_virtual in ("fortran-rt", gfortran_str):
+            pkg("*").depends_on(
+                fortran_virtual,
+                when=f"^[virtuals=fortran deptypes=build] {spec.name}@{spec.versions}",
+                type="link",
+                description=f"Add a dependency on '{gfortran_str}' for nodes compiled with "
+                f"{str(spec)} and using the 'fortran' language",
+            )
+        # The version of gcc-runtime is the same as the %gcc used to "compile" it
+        pkg("gcc-runtime").requires(
+            f"@{str(spec.versions)}", when=f"^[deptypes=build] {spec.name}@{spec.versions}"
+        )
+
+        # If a node used %gcc@X.Y its dependencies must use gcc-runtime@:X.Y
+        # (technically @:X is broader than ... <= @=X but this should work in practice)
+        pkg("*").propagate(
+            f"gcc@:{str(spec.version)}", when=f"^[deptypes=build] {spec.name}@{spec.versions}"
+        )
diff --git a/var/spack/repos/builtin.mock/packages/intel-oneapi-compilers/package.py b/var/spack/repos/builtin.mock/packages/intel-oneapi-compilers/package.py
index 892a09f7dc0..23515fcb26b 100644
--- a/var/spack/repos/builtin.mock/packages/intel-oneapi-compilers/package.py
+++ b/var/spack/repos/builtin.mock/packages/intel-oneapi-compilers/package.py
@@ -27,6 +27,8 @@ class IntelOneapiCompilers(Package, CompilerPackage):
         r"(?:(?:oneAPI DPC\+\+(?:\/C\+\+)? Compiler)|(?:\(IFORT\))|(?:\(IFX\))) (\S+)"
     )
 
+    depends_on("c", type="build")
+
     @property
     def compiler_search_prefix(self):
         return self.prefix.foo.bar.baz.bin
diff --git a/var/spack/repos/builtin.mock/packages/libdwarf/package.py b/var/spack/repos/builtin.mock/packages/libdwarf/package.py
index 1d460bc6ea6..faf8e4e6513 100644
--- a/var/spack/repos/builtin.mock/packages/libdwarf/package.py
+++ b/var/spack/repos/builtin.mock/packages/libdwarf/package.py
@@ -21,5 +21,8 @@ class Libdwarf(Package):
 
     depends_on("libelf")
 
+    depends_on("c", type="build")
+    depends_on("cxx", type="build")
+
     def install(self, spec, prefix):
         touch(prefix.libdwarf)
diff --git a/var/spack/repos/builtin.mock/packages/libelf/package.py b/var/spack/repos/builtin.mock/packages/libelf/package.py
index 31b46164ae2..0f3bd4dd68d 100644
--- a/var/spack/repos/builtin.mock/packages/libelf/package.py
+++ b/var/spack/repos/builtin.mock/packages/libelf/package.py
@@ -16,5 +16,7 @@ class Libelf(Package):
 
     patch("local.patch", when="@0.8.10")
 
+    depends_on("c", type="build")
+
     def install(self, spec, prefix):
         touch(prefix.libelf)
diff --git a/var/spack/repos/builtin.mock/packages/llvm/package.py b/var/spack/repos/builtin.mock/packages/llvm/package.py
index 135ae5ebfa2..710c87e45f5 100644
--- a/var/spack/repos/builtin.mock/packages/llvm/package.py
+++ b/var/spack/repos/builtin.mock/packages/llvm/package.py
@@ -18,6 +18,8 @@ class Llvm(Package, CompilerPackage):
         "clang", default=True, description="Build the LLVM C/C++/Objective-C compiler frontend"
     )
 
+    provides("c", "cxx", when="+clang")
+
     c_names = ["clang"]
     cxx_names = ["clang++"]
     fortran_names = ["flang"]
@@ -28,3 +30,36 @@ def install(self, spec, prefix):
         with open(prefix.bin.gcc, "w") as f:
             f.write('#!/bin/bash\necho "%s"' % str(spec.version))
         set_executable(prefix.bin.gcc)
+
+    @property
+    def cc(self):
+        msg = "cannot retrieve C compiler [spec is not concrete]"
+        assert self.spec.concrete, msg
+        if self.spec.external:
+            return self.spec.extra_attributes["compilers"].get("c", None)
+        result = None
+        if "+clang" in self.spec:
+            result = os.path.join(self.spec.prefix.bin, "clang")
+        return result
+
+    @property
+    def cxx(self):
+        msg = "cannot retrieve C++ compiler [spec is not concrete]"
+        assert self.spec.concrete, msg
+        if self.spec.external:
+            return self.spec.extra_attributes["compilers"].get("cxx", None)
+        result = None
+        if "+clang" in self.spec:
+            result = os.path.join(self.spec.prefix.bin, "clang++")
+        return result
+
+    @property
+    def fortan(self):
+        msg = "cannot retrieve Fortran compiler [spec is not concrete]"
+        assert self.spec.concrete, msg
+        if self.spec.external:
+            return self.spec.extra_attributes["compilers"].get("fc", None)
+        result = None
+        if "+flang" in self.spec:
+            result = os.path.join(self.spec.prefix.bin, "flang")
+        return result
diff --git a/var/spack/repos/builtin.mock/packages/mpich/package.py b/var/spack/repos/builtin.mock/packages/mpich/package.py
index 4c87d6932fa..3163143c4d4 100644
--- a/var/spack/repos/builtin.mock/packages/mpich/package.py
+++ b/var/spack/repos/builtin.mock/packages/mpich/package.py
@@ -27,6 +27,10 @@ class Mpich(Package):
     provides("mpi@:3", when="@3:")
     provides("mpi@:1", when="@:1")
 
+    depends_on("c", type="build")
+    depends_on("cxx", type="build")
+    depends_on("fortran", type="build")
+
     def install(self, spec, prefix):
         touch(prefix.mpich)
 
diff --git a/var/spack/repos/builtin.mock/packages/mpich2/package.py b/var/spack/repos/builtin.mock/packages/mpich2/package.py
index 51b78de2a3d..f175a7c7b2a 100644
--- a/var/spack/repos/builtin.mock/packages/mpich2/package.py
+++ b/var/spack/repos/builtin.mock/packages/mpich2/package.py
@@ -25,6 +25,8 @@ class Mpich2(Package):
     provides("mpi@:2.1", when="@1.1:")
     provides("mpi@:2.2", when="@1.2:")
 
+    depends_on("c", type="build")
+
     def install(self, spec, prefix):
         configure("--prefix=%s" % prefix)
         make()
diff --git a/var/spack/repos/builtin.mock/packages/mpileaks/package.py b/var/spack/repos/builtin.mock/packages/mpileaks/package.py
index fe1c6adce9c..e731ecc7f2e 100644
--- a/var/spack/repos/builtin.mock/packages/mpileaks/package.py
+++ b/var/spack/repos/builtin.mock/packages/mpileaks/package.py
@@ -25,6 +25,8 @@ class Mpileaks(Package):
     depends_on("mpi")
     depends_on("callpath")
 
+    depends_on("c", type="build")
+
     # Will be used to try raising an exception
     libs = None
 
diff --git a/var/spack/repos/builtin.mock/packages/multimethod/package.py b/var/spack/repos/builtin.mock/packages/multimethod/package.py
index ad58ffd8190..e45758a6160 100644
--- a/var/spack/repos/builtin.mock/packages/multimethod/package.py
+++ b/var/spack/repos/builtin.mock/packages/multimethod/package.py
@@ -26,6 +26,7 @@ class Multimethod(MultimethodBase):
     variant("mpi", default=False, description="")
 
     depends_on("mpi", when="+mpi")
+    depends_on("c", type="build")
 
     #
     # These functions are only valid for versions 1, 3, and 4.
@@ -80,7 +81,7 @@ def mpi_version(self):
     def has_a_default(self):
         return "default"
 
-    @when("%gcc")
+    @when("%gcc@10:")
     def has_a_default(self):
         return "gcc"
 
diff --git a/var/spack/repos/builtin.mock/packages/multivalue-variant/package.py b/var/spack/repos/builtin.mock/packages/multivalue-variant/package.py
index 70ab8306a30..8cf18793cd0 100644
--- a/var/spack/repos/builtin.mock/packages/multivalue-variant/package.py
+++ b/var/spack/repos/builtin.mock/packages/multivalue-variant/package.py
@@ -42,3 +42,5 @@ class MultivalueVariant(Package):
     depends_on("callpath")
     depends_on("pkg-a")
     depends_on("pkg-a@1.0", when="fee=barbaz")
+
+    depends_on("c", type="build")
diff --git a/var/spack/repos/builtin.mock/packages/openblas-with-lapack/package.py b/var/spack/repos/builtin.mock/packages/openblas-with-lapack/package.py
index fa275b8d386..441e7d67a7c 100644
--- a/var/spack/repos/builtin.mock/packages/openblas-with-lapack/package.py
+++ b/var/spack/repos/builtin.mock/packages/openblas-with-lapack/package.py
@@ -15,3 +15,4 @@ class OpenblasWithLapack(Package):
     version("0.2.15", md5="b1190f3d3471685f17cfd1ec1d252ac9")
 
     provides("lapack", "blas")
+    depends_on("c", type="build")
diff --git a/var/spack/repos/builtin.mock/packages/openblas/package.py b/var/spack/repos/builtin.mock/packages/openblas/package.py
index d6ecb25019a..1e8b67cd7b3 100644
--- a/var/spack/repos/builtin.mock/packages/openblas/package.py
+++ b/var/spack/repos/builtin.mock/packages/openblas/package.py
@@ -19,6 +19,8 @@ class Openblas(Package):
 
     variant("shared", default=True, description="Build shared libraries")
 
+    depends_on("c", type="build")
+
     # See #20019 for this conflict
     conflicts("%gcc@:4.4", when="@0.2.14:")
 
diff --git a/var/spack/repos/builtin.mock/packages/pkg-a/package.py b/var/spack/repos/builtin.mock/packages/pkg-a/package.py
index 646172b778d..cd461d75871 100644
--- a/var/spack/repos/builtin.mock/packages/pkg-a/package.py
+++ b/var/spack/repos/builtin.mock/packages/pkg-a/package.py
@@ -36,6 +36,8 @@ class PkgA(AutotoolsPackage):
     depends_on("pkg-b", when="foobar=bar")
     depends_on("test-dependency", type="test")
 
+    depends_on("c", type="build")
+
     parallel = False
 
 
diff --git a/var/spack/repos/builtin.mock/packages/pkg-b/package.py b/var/spack/repos/builtin.mock/packages/pkg-b/package.py
index bc20950ebc5..40bd6189dbc 100644
--- a/var/spack/repos/builtin.mock/packages/pkg-b/package.py
+++ b/var/spack/repos/builtin.mock/packages/pkg-b/package.py
@@ -19,4 +19,5 @@ class PkgB(Package):
         "foo", description="", values=any_combination_of("bar", "baz", "fee").with_default("bar")
     )
 
+    depends_on("c", type="build")
     depends_on("test-dependency", type="test")
diff --git a/var/spack/repos/builtin.mock/packages/pkg-c/package.py b/var/spack/repos/builtin.mock/packages/pkg-c/package.py
index 2c76f1ea518..0501a49ed79 100644
--- a/var/spack/repos/builtin.mock/packages/pkg-c/package.py
+++ b/var/spack/repos/builtin.mock/packages/pkg-c/package.py
@@ -12,4 +12,16 @@ class PkgC(Package):
     homepage = "http://www.example.com"
     url = "http://www.example.com/c-1.0.tar.gz"
 
+    # Needed to test CDash reporting
+    phases = ["configure", "build", "install"]
+
     version("1.0", md5="0123456789abcdef0123456789abcdef")
+
+    def configure(self, spec, prefix):
+        pass
+
+    def build(self, spec, prefix):
+        pass
+
+    def install(self, spec, prefix):
+        touch(prefix.pkg_c)
diff --git a/var/spack/repos/builtin.mock/packages/requires_clang/package.py b/var/spack/repos/builtin.mock/packages/requires_clang/package.py
index 72c00e6c104..fabc4f4364d 100644
--- a/var/spack/repos/builtin.mock/packages/requires_clang/package.py
+++ b/var/spack/repos/builtin.mock/packages/requires_clang/package.py
@@ -16,3 +16,5 @@ class RequiresClang(Package):
     version("0.9", md5="abcd456789abcdef0123456789abcdef")
 
     requires("%clang", msg="can only be compiled with Clang")
+
+    depends_on("c", type="build")
diff --git a/var/spack/repos/builtin.mock/packages/requires_clang_or_gcc/package.py b/var/spack/repos/builtin.mock/packages/requires_clang_or_gcc/package.py
index 956f025e4cf..ac64223626e 100644
--- a/var/spack/repos/builtin.mock/packages/requires_clang_or_gcc/package.py
+++ b/var/spack/repos/builtin.mock/packages/requires_clang_or_gcc/package.py
@@ -16,3 +16,5 @@ class RequiresClangOrGcc(Package):
     version("0.9", md5="abcd456789abcdef0123456789abcdef")
 
     requires("%gcc", "%clang", policy="one_of")
+
+    depends_on("c", type="build")
diff --git a/var/spack/repos/builtin.mock/packages/simple-inheritance/package.py b/var/spack/repos/builtin.mock/packages/simple-inheritance/package.py
index 67042671bab..1111932410c 100644
--- a/var/spack/repos/builtin.mock/packages/simple-inheritance/package.py
+++ b/var/spack/repos/builtin.mock/packages/simple-inheritance/package.py
@@ -29,3 +29,5 @@ class SimpleInheritance(BaseWithDirectives):
 
     depends_on("openblas", when="+openblas")
     provides("lapack", when="+openblas")
+
+    depends_on("c", type="build")
diff --git a/var/spack/repos/builtin.mock/packages/sticky-variant-dependent/package.py b/var/spack/repos/builtin.mock/packages/sticky-variant-dependent/package.py
index 2036a31b287..8ad5bc8baf2 100644
--- a/var/spack/repos/builtin.mock/packages/sticky-variant-dependent/package.py
+++ b/var/spack/repos/builtin.mock/packages/sticky-variant-dependent/package.py
@@ -15,3 +15,5 @@ class StickyVariantDependent(AutotoolsPackage):
 
     depends_on("sticky-variant")
     conflicts("%gcc", when="^sticky-variant~allow-gcc")
+
+    depends_on("c", type="build")
diff --git a/var/spack/repos/builtin.mock/packages/sticky-variant/package.py b/var/spack/repos/builtin.mock/packages/sticky-variant/package.py
index 334b4a44bbe..7b868078647 100644
--- a/var/spack/repos/builtin.mock/packages/sticky-variant/package.py
+++ b/var/spack/repos/builtin.mock/packages/sticky-variant/package.py
@@ -16,3 +16,5 @@ class StickyVariant(AutotoolsPackage):
     variant("allow-gcc", description="", default=False, sticky=True)
 
     conflicts("%gcc", when="~allow-gcc")
+
+    depends_on("c", type="build")
diff --git a/var/spack/repos/builtin.mock/packages/with-constraint-met/package.py b/var/spack/repos/builtin.mock/packages/with-constraint-met/package.py
index e300a0fd366..3b5fe58faa5 100644
--- a/var/spack/repos/builtin.mock/packages/with-constraint-met/package.py
+++ b/var/spack/repos/builtin.mock/packages/with-constraint-met/package.py
@@ -15,6 +15,8 @@ class WithConstraintMet(Package):
     version("2.0", md5="0123456789abcdef0123456789abcdef")
     version("1.0", md5="0123456789abcdef0123456789abcdef")
 
+    depends_on("c", type="build")
+
     with when("@1.0"):
         depends_on("pkg-b")
         conflicts("%gcc", when="+foo")
diff --git a/var/spack/repos/builtin.mock/packages/zmpi/package.py b/var/spack/repos/builtin.mock/packages/zmpi/package.py
index 0a71e47038e..1c9e7d970c4 100644
--- a/var/spack/repos/builtin.mock/packages/zmpi/package.py
+++ b/var/spack/repos/builtin.mock/packages/zmpi/package.py
@@ -16,4 +16,6 @@ class Zmpi(Package):
     version("1.0", md5="0123456789abcdef0123456789abcdef")
 
     provides("mpi@:10.0")
+
     depends_on("fake")
+    depends_on("c", type="build")
diff --git a/var/spack/repos/compiler_runtime.test/packages/gcc b/var/spack/repos/compiler_runtime.test/packages/gcc
new file mode 120000
index 00000000000..17e3b9e575b
--- /dev/null
+++ b/var/spack/repos/compiler_runtime.test/packages/gcc
@@ -0,0 +1 @@
+../../builtin.mock/packages/gcc
\ No newline at end of file
diff --git a/var/spack/repos/compiler_runtime.test/packages/gcc-runtime b/var/spack/repos/compiler_runtime.test/packages/gcc-runtime
new file mode 120000
index 00000000000..0e770302bf4
--- /dev/null
+++ b/var/spack/repos/compiler_runtime.test/packages/gcc-runtime
@@ -0,0 +1 @@
+../../builtin.mock/packages/gcc-runtime
\ No newline at end of file
diff --git a/var/spack/repos/compiler_runtime.test/packages/gcc-runtime/package.py b/var/spack/repos/compiler_runtime.test/packages/gcc-runtime/package.py
deleted file mode 100644
index a522afce835..00000000000
--- a/var/spack/repos/compiler_runtime.test/packages/gcc-runtime/package.py
+++ /dev/null
@@ -1,13 +0,0 @@
-# Copyright 2013-2024 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 GccRuntime(Package):
-    homepage = "https://example.com"
-    has_code = False
-    tags = ["runtime"]
-    requires("%gcc")
diff --git a/var/spack/repos/compiler_runtime.test/packages/gcc/package.py b/var/spack/repos/compiler_runtime.test/packages/gcc/package.py
deleted file mode 100644
index 6d8decc26b2..00000000000
--- a/var/spack/repos/compiler_runtime.test/packages/gcc/package.py
+++ /dev/null
@@ -1,35 +0,0 @@
-# Copyright 2013-2024 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 Gcc(Package):
-    homepage = "http://www.example.com/"
-    has_code = False
-
-    version("13.2.0")
-    version("12.3.0")
-
-    @classmethod
-    def runtime_constraints(cls, *, spec, pkg):
-        pkg("*").depends_on(
-            "gcc-runtime",
-            when="%gcc",
-            type="link",
-            description="If any package uses %gcc, it depends on gcc-runtime",
-        )
-        pkg("*").depends_on(
-            f"gcc-runtime@{str(spec.version)}:",
-            when=f"%{str(spec)}",
-            type="link",
-            description=f"If any package uses %{str(spec)}, "
-            f"it depends on gcc-runtime@{str(spec.version)}:",
-        )
-        # The version of gcc-runtime is the same as the %gcc used to "compile" it
-        pkg("gcc-runtime").requires(f"@={str(spec.version)}", when=f"%{str(spec)}")
-
-        # If a node used %gcc@X.Y its dependencies must use gcc-runtime@:X.Y
-        pkg("*").propagate(f"%gcc@:{str(spec.version)}", when=f"%{str(spec)}")
diff --git a/var/spack/repos/compiler_runtime.test/packages/glibc b/var/spack/repos/compiler_runtime.test/packages/glibc
new file mode 120000
index 00000000000..f9cd337cc24
--- /dev/null
+++ b/var/spack/repos/compiler_runtime.test/packages/glibc
@@ -0,0 +1 @@
+../../builtin.mock/packages/glibc
\ No newline at end of file
diff --git a/var/spack/repos/compiler_runtime.test/packages/gmake b/var/spack/repos/compiler_runtime.test/packages/gmake
new file mode 120000
index 00000000000..86bbc3dd35a
--- /dev/null
+++ b/var/spack/repos/compiler_runtime.test/packages/gmake
@@ -0,0 +1 @@
+../../builtin.mock/packages/gmake
\ No newline at end of file
diff --git a/var/spack/repos/compiler_runtime.test/packages/gnuconfig b/var/spack/repos/compiler_runtime.test/packages/gnuconfig
new file mode 120000
index 00000000000..cfb894494bd
--- /dev/null
+++ b/var/spack/repos/compiler_runtime.test/packages/gnuconfig
@@ -0,0 +1 @@
+../../builtin.mock/packages/gnuconfig
\ No newline at end of file
diff --git a/var/spack/repos/compiler_runtime.test/packages/pkg-a/package.py b/var/spack/repos/compiler_runtime.test/packages/pkg-a/package.py
index 22ca47cf130..0de1032a54a 100644
--- a/var/spack/repos/compiler_runtime.test/packages/pkg-a/package.py
+++ b/var/spack/repos/compiler_runtime.test/packages/pkg-a/package.py
@@ -11,3 +11,6 @@ class PkgA(Package):
 
     version("1.0")
     depends_on("pkg-b")
+
+    depends_on("c", type="build")
+    depends_on("cxx", type="build")
diff --git a/var/spack/repos/compiler_runtime.test/packages/pkg-b/package.py b/var/spack/repos/compiler_runtime.test/packages/pkg-b/package.py
index d4147bf47e6..9e8fb07c18e 100644
--- a/var/spack/repos/compiler_runtime.test/packages/pkg-b/package.py
+++ b/var/spack/repos/compiler_runtime.test/packages/pkg-b/package.py
@@ -10,3 +10,6 @@ class PkgB(Package):
     has_code = False
 
     version("1.0")
+
+    depends_on("c", type="build")
+    depends_on("cxx", type="build")
diff --git a/var/spack/repos/duplicates.test/packages/gcc b/var/spack/repos/duplicates.test/packages/gcc
new file mode 120000
index 00000000000..17e3b9e575b
--- /dev/null
+++ b/var/spack/repos/duplicates.test/packages/gcc
@@ -0,0 +1 @@
+../../builtin.mock/packages/gcc
\ No newline at end of file
diff --git a/var/spack/repos/duplicates.test/packages/gcc-runtime b/var/spack/repos/duplicates.test/packages/gcc-runtime
new file mode 120000
index 00000000000..0e770302bf4
--- /dev/null
+++ b/var/spack/repos/duplicates.test/packages/gcc-runtime
@@ -0,0 +1 @@
+../../builtin.mock/packages/gcc-runtime
\ No newline at end of file
diff --git a/var/spack/repos/duplicates.test/packages/glibc b/var/spack/repos/duplicates.test/packages/glibc
new file mode 120000
index 00000000000..f9cd337cc24
--- /dev/null
+++ b/var/spack/repos/duplicates.test/packages/glibc
@@ -0,0 +1 @@
+../../builtin.mock/packages/glibc
\ No newline at end of file
diff --git a/var/spack/repos/duplicates.test/packages/gnuconfig b/var/spack/repos/duplicates.test/packages/gnuconfig
new file mode 120000
index 00000000000..cfb894494bd
--- /dev/null
+++ b/var/spack/repos/duplicates.test/packages/gnuconfig
@@ -0,0 +1 @@
+../../builtin.mock/packages/gnuconfig
\ No newline at end of file
diff --git a/var/spack/repos/duplicates.test/packages/llvm b/var/spack/repos/duplicates.test/packages/llvm
new file mode 120000
index 00000000000..7dddef3a960
--- /dev/null
+++ b/var/spack/repos/duplicates.test/packages/llvm
@@ -0,0 +1 @@
+../../builtin.mock/packages/llvm
\ No newline at end of file
diff --git a/var/spack/repos/edges.test/packages/gcc b/var/spack/repos/edges.test/packages/gcc
new file mode 120000
index 00000000000..17e3b9e575b
--- /dev/null
+++ b/var/spack/repos/edges.test/packages/gcc
@@ -0,0 +1 @@
+../../builtin.mock/packages/gcc
\ No newline at end of file
diff --git a/var/spack/repos/edges.test/packages/gcc-runtime b/var/spack/repos/edges.test/packages/gcc-runtime
new file mode 120000
index 00000000000..0e770302bf4
--- /dev/null
+++ b/var/spack/repos/edges.test/packages/gcc-runtime
@@ -0,0 +1 @@
+../../builtin.mock/packages/gcc-runtime
\ No newline at end of file
diff --git a/var/spack/repos/edges.test/packages/glibc b/var/spack/repos/edges.test/packages/glibc
new file mode 120000
index 00000000000..f9cd337cc24
--- /dev/null
+++ b/var/spack/repos/edges.test/packages/glibc
@@ -0,0 +1 @@
+../../builtin.mock/packages/glibc
\ No newline at end of file
diff --git a/var/spack/repos/edges.test/packages/gmake b/var/spack/repos/edges.test/packages/gmake
new file mode 120000
index 00000000000..86bbc3dd35a
--- /dev/null
+++ b/var/spack/repos/edges.test/packages/gmake
@@ -0,0 +1 @@
+../../builtin.mock/packages/gmake
\ No newline at end of file
diff --git a/var/spack/repos/edges.test/packages/gnuconfig b/var/spack/repos/edges.test/packages/gnuconfig
new file mode 120000
index 00000000000..cfb894494bd
--- /dev/null
+++ b/var/spack/repos/edges.test/packages/gnuconfig
@@ -0,0 +1 @@
+../../builtin.mock/packages/gnuconfig
\ No newline at end of file
diff --git a/var/spack/repos/edges.test/packages/llvm b/var/spack/repos/edges.test/packages/llvm
new file mode 120000
index 00000000000..7dddef3a960
--- /dev/null
+++ b/var/spack/repos/edges.test/packages/llvm
@@ -0,0 +1 @@
+../../builtin.mock/packages/llvm
\ No newline at end of file
diff --git a/var/spack/repos/flags.test/packages/gcc b/var/spack/repos/flags.test/packages/gcc
new file mode 120000
index 00000000000..17e3b9e575b
--- /dev/null
+++ b/var/spack/repos/flags.test/packages/gcc
@@ -0,0 +1 @@
+../../builtin.mock/packages/gcc
\ No newline at end of file
diff --git a/var/spack/repos/flags.test/packages/gcc-runtime b/var/spack/repos/flags.test/packages/gcc-runtime
new file mode 120000
index 00000000000..0e770302bf4
--- /dev/null
+++ b/var/spack/repos/flags.test/packages/gcc-runtime
@@ -0,0 +1 @@
+../../builtin.mock/packages/gcc-runtime
\ No newline at end of file
diff --git a/var/spack/repos/flags.test/packages/glibc b/var/spack/repos/flags.test/packages/glibc
new file mode 120000
index 00000000000..f9cd337cc24
--- /dev/null
+++ b/var/spack/repos/flags.test/packages/glibc
@@ -0,0 +1 @@
+../../builtin.mock/packages/glibc
\ No newline at end of file
diff --git a/var/spack/repos/flags.test/packages/gmake b/var/spack/repos/flags.test/packages/gmake
new file mode 120000
index 00000000000..eefeb5e47f5
--- /dev/null
+++ b/var/spack/repos/flags.test/packages/gmake
@@ -0,0 +1 @@
+../../builtin.mock/packages/gmake/
\ No newline at end of file
diff --git a/var/spack/repos/flags.test/packages/gnuconfig b/var/spack/repos/flags.test/packages/gnuconfig
new file mode 120000
index 00000000000..cfb894494bd
--- /dev/null
+++ b/var/spack/repos/flags.test/packages/gnuconfig
@@ -0,0 +1 @@
+../../builtin.mock/packages/gnuconfig
\ No newline at end of file
diff --git a/var/spack/repos/requirements.test/packages/gcc b/var/spack/repos/requirements.test/packages/gcc
new file mode 120000
index 00000000000..17e3b9e575b
--- /dev/null
+++ b/var/spack/repos/requirements.test/packages/gcc
@@ -0,0 +1 @@
+../../builtin.mock/packages/gcc
\ No newline at end of file
diff --git a/var/spack/repos/requirements.test/packages/gcc-runtime b/var/spack/repos/requirements.test/packages/gcc-runtime
new file mode 120000
index 00000000000..0e770302bf4
--- /dev/null
+++ b/var/spack/repos/requirements.test/packages/gcc-runtime
@@ -0,0 +1 @@
+../../builtin.mock/packages/gcc-runtime
\ No newline at end of file
diff --git a/var/spack/repos/requirements.test/packages/glibc b/var/spack/repos/requirements.test/packages/glibc
new file mode 120000
index 00000000000..f9cd337cc24
--- /dev/null
+++ b/var/spack/repos/requirements.test/packages/glibc
@@ -0,0 +1 @@
+../../builtin.mock/packages/glibc
\ No newline at end of file
diff --git a/var/spack/repos/requirements.test/packages/gmake b/var/spack/repos/requirements.test/packages/gmake
new file mode 120000
index 00000000000..eefeb5e47f5
--- /dev/null
+++ b/var/spack/repos/requirements.test/packages/gmake
@@ -0,0 +1 @@
+../../builtin.mock/packages/gmake/
\ No newline at end of file
diff --git a/var/spack/repos/requirements.test/packages/gnuconfig b/var/spack/repos/requirements.test/packages/gnuconfig
new file mode 120000
index 00000000000..cfb894494bd
--- /dev/null
+++ b/var/spack/repos/requirements.test/packages/gnuconfig
@@ -0,0 +1 @@
+../../builtin.mock/packages/gnuconfig
\ No newline at end of file
diff --git a/var/spack/repos/requirements.test/packages/llvm b/var/spack/repos/requirements.test/packages/llvm
new file mode 120000
index 00000000000..7dddef3a960
--- /dev/null
+++ b/var/spack/repos/requirements.test/packages/llvm
@@ -0,0 +1 @@
+../../builtin.mock/packages/llvm
\ No newline at end of file