Fix hash handling in directory layout
- Currently, build dependencies are not currently hashed; we are waiting to hash these until we have smarter concretization that can reuse more installed specs. The layout needs to account for this when checking whethert things are installed.
This commit is contained in:
parent
f5bc0cbb65
commit
69d45b49e9
1 changed files with 7 additions and 0 deletions
|
@ -268,6 +268,13 @@ def check_installed(self, spec):
|
|||
if installed_spec == spec:
|
||||
return path
|
||||
|
||||
# DAG hashes currently do not include build dependencies.
|
||||
#
|
||||
# TODO: remove this when we do better concretization and don't
|
||||
# ignore build-only deps in hashes.
|
||||
elif installed_spec == spec.copy(deps=('link', 'run')):
|
||||
return path
|
||||
|
||||
if spec.dag_hash() == installed_spec.dag_hash():
|
||||
raise SpecHashCollisionError(spec, installed_spec)
|
||||
else:
|
||||
|
|
Loading…
Reference in a new issue