tweak presentation of build cache by recipe

Author: bcumming

stackinator/builder.py

@@ -233,7 +233,7 @@ def generate(self, recipe):
                     spack_version=spack_version,
                     spack_meta=spack_meta,
                     gpg_keys=recipe.mirrors.keys,
-                    cache=recipe.mirrors.buildcache,
+                    cache=recipe.build_cache_mirror,
                     exclude_from_cache=["nvhpc", "cuda", "perl"],
                     verbose=False,
                 )

stackinator/main.py

@@ -86,7 +86,7 @@ def make_argparser():
         "--build",
         required=True,
         type=str,
-        help="Where to set up the stackinator build directory. ('/tmp' is not allowed, use '/var/tmp'",
+        help="Where to set up the stackinator build directory. ('/tmp' is not allowed, use '/var/tmp')",
     )
     parser.add_argument("--no-bwrap", action="store_true", required=False)
     parser.add_argument(

stackinator/recipe.py

@@ -172,8 +172,7 @@ def __init__(self, args):
         # load the optional mirrors.yaml from system config, and add any additional
         # mirrors specified on the command line.
         self._logger.debug("Configuring mirrors.")
-        self.mirrors = mirror.Mirrors(self.system_config_path, args.cache)
-        self.cache = self.mirrors.build_cache_mirror
+        self.mirrors = mirror.Mirrors(self.system_config_path, pathlib.Path(args.cache))
 
         # optional post install hook
         if self.post_install_hook is not None:
@@ -202,6 +201,13 @@ def spack_repo(self):
             return repo_path
         return None
 
+    # Returns:
+    #   Path: if the recipe specified a build cache mirror
+    #   None: if no build cache mirror is used
+    @property
+    def build_cache_mirror(self):
+        return self.mirrors.build_cache_mirror
+
     # Returns:
     #   Path: of the recipe extra path if it exists
     #   None: if there is no user-provided extra path in the recipe
@@ -511,7 +517,7 @@ def compiler_files(self):
         )
 
         makefile_template = env.get_template("Makefile.compilers")
-        push_to_cache = self.cache
+        push_to_cache = self.build_cache_mirror is not None
         files["makefile"] = makefile_template.render(
             compilers=self.compilers,
             push_to_cache=push_to_cache,
@@ -542,7 +548,7 @@ def environment_files(self):
         jenv.filters["py2yaml"] = schema.py2yaml
 
         makefile_template = jenv.get_template("Makefile.environments")
-        push_to_cache = self.cache is not None
+        push_to_cache = self.build_cache_mirror is not None
         files["makefile"] = makefile_template.render(
             environments=self.environments,
             push_to_cache=push_to_cache,