remove completed TODO

This commit is contained in:
Gregory Becker 2024-08-19 14:39:16 -07:00
parent 930dfd7cc4
commit af4cb0a14b

View File

@ -715,7 +715,6 @@ def get_buildfile_manifest(spec):
def hashes_to_prefixes(spec): def hashes_to_prefixes(spec):
"""Return a dictionary of hashes to prefixes for a spec and its deps, excluding externals""" """Return a dictionary of hashes to prefixes for a spec and its deps, excluding externals"""
# TODO: Get this to return a tuple value (name, prefix) and modify all callers.
return { return {
s.dag_hash(): (s.name, str(s.prefix)) s.dag_hash(): (s.name, str(s.prefix))
for s in itertools.chain( for s in itertools.chain(
@ -1532,7 +1531,6 @@ def _push_oci(
Dict[str, spack.oci.oci.Blob], Dict[str, spack.oci.oci.Blob],
List[Tuple[Spec, BaseException]], List[Tuple[Spec, BaseException]],
]: ]:
# Spec dag hash -> blob # Spec dag hash -> blob
checksums: Dict[str, spack.oci.oci.Blob] = {} checksums: Dict[str, spack.oci.oci.Blob] = {}
@ -2119,7 +2117,6 @@ def relocate_package(spec):
# This is a new dependency # This is a new dependency
tty.debug(f"{spec} does not have relocation for {name}") tty.debug(f"{spec} does not have relocation for {name}")
if lookup_dag_hash in hash_to_old_prefix: if lookup_dag_hash in hash_to_old_prefix:
old_dep_prefix = hash_to_old_prefix[lookup_dag_hash] old_dep_prefix = hash_to_old_prefix[lookup_dag_hash]
prefix_to_prefix_bin[old_dep_prefix] = new_dep_prefix prefix_to_prefix_bin[old_dep_prefix] = new_dep_prefix