repo: don't return builtin path plus docstring tweaks and TODOs

lib/spack/spack/bootstrap/clingo.py

@@ -146,6 +146,7 @@ def libc_external_spec(self) -> "spack.spec.Spec":
         return self._external_spec(result)
 
     def _external_spec(self, initial_spec) -> "spack.spec.Spec":
+        #TODO: Does this need to be changed?
         initial_spec.namespace = "builtin"
         initial_spec.compiler = self.host_compiler.spec
         initial_spec.architecture = self.host_architecture

lib/spack/spack/package_base.py

@@ -958,6 +958,7 @@ def fullnames(cls):
             namespace = getattr(cls, "namespace", None)
             if namespace:
                 fullnames.append("%s.%s" % (namespace, cls.name))
+            #TODO: Does this need to be changed?
             if namespace == "builtin":
                 # builtin packages cannot inherit from other repos
                 break

lib/spack/spack/paths.py

@@ -58,7 +58,10 @@
 
 # read-only things in $spack/var/spack
 repos_path = os.path.join(var_path, "repos")
+
+#TODO: How do we want to replace this?
 packages_path = os.path.join(repos_path, "builtin")
+
 mock_packages_path = os.path.join(repos_path, "builtin.mock")
 
 #

lib/spack/spack/repo.py

@@ -57,7 +57,7 @@ def python_package_for_repo(namespace):
 
     For instance:
 
-        python_package_for_repo('builtin') == 'spack.pkg.builtin'
+        python_package_for_repo('builtin.mock') == 'spack.pkg.builtin.mock'
 
     Args:
         namespace (str): repo namespace
@@ -70,7 +70,7 @@ def namespace_from_fullname(fullname):
 
     For instance:
 
-        namespace_from_fullname('spack.pkg.builtin.hdf5') == 'builtin'
+        namespace_from_fullname('spack.pkg.builtin.mock.hdf5') == 'builtin.mock'
 
     Args:
         fullname (str): full name for the Python module
@@ -183,7 +183,10 @@ def packages_path():
     try:
         return PATH.get_repo("builtin.mock").packages_path
     except UnknownNamespaceError:
-        return PATH.get_repo("builtin").packages_path
+        #TODO: Replace this with the approach needed to ensure using the new
+        #TODO: built-in repository option
+        #return PATH.get_repo("builtin").packages_path
+        return None
 
 
 class GitExe:

lib/spack/spack/spec.py

@@ -2266,6 +2266,7 @@ def to_node_dict(self, hash=ht.dag_hash):
             )
         return d
 
+    #TODO: Does the example namespace need to change from builtin?
     def to_dict(self, hash=ht.dag_hash):
         """Create a dictionary suitable for writing this spec to YAML or JSON.