Skip to content
Snippets Groups Projects
Commit ef2c42a8 authored by Barry Smith's avatar Barry Smith Committed by Todd Gamblin
Browse files

Do not check directory layout for external packages (#2172)

External packages do not have an spec.yaml file so don't check for it.
Without this change any time a package depends on an external package
when the new package is installed you will get the error

Install prefix exists but contains no spec.yaml

This problem has also haunted me since I started using Spack since PETSc
depends on Python and I used an external python but fortunately it
was relatively easy to debug once I could reproduce it at will.

Funded-by: IDEAS
Project: IDEAS/xSDK
Time: 1 hour
parent c2d58d39
No related branches found
No related tags found
No related merge requests found
...@@ -468,7 +468,7 @@ def _add(self, spec, directory_layout=None, explicit=False): ...@@ -468,7 +468,7 @@ def _add(self, spec, directory_layout=None, explicit=False):
if key not in self._data: if key not in self._data:
installed = False installed = False
path = None path = None
if directory_layout: if not spec.external and directory_layout:
path = directory_layout.path_for_spec(spec) path = directory_layout.path_for_spec(spec)
try: try:
directory_layout.check_installed(spec) directory_layout.check_installed(spec)
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment