diff --git a/lib/spack/spack/spec.py b/lib/spack/spack/spec.py index 047764c8aef..cc2d5ca5522 100644 --- a/lib/spack/spack/spec.py +++ b/lib/spack/spack/spec.py @@ -2330,6 +2330,10 @@ def concretize(self, tests=False): # TODO: internal configuration conflicts within one package. matches = [] for x in self.traverse(): + if x.external: + # external specs are already built, don't worry about whether + # it's possible to build that configuration with Spack + continue for conflict_spec, when_list in x.package_class.conflicts.items(): if x.satisfies(conflict_spec, strict=True): for when_spec, msg in when_list: diff --git a/lib/spack/spack/test/concretize.py b/lib/spack/spack/test/concretize.py index a7a89cc78d8..94770ad5f84 100644 --- a/lib/spack/spack/test/concretize.py +++ b/lib/spack/spack/test/concretize.py @@ -500,6 +500,12 @@ def test_conflicts_in_spec(self, conflict_spec): with pytest.raises(exc_type): s.concretize() + def test_no_conflixt_in_external_specs(self, conflict_spec): + # clear deps because external specs cannot depend on anything + ext = Spec(conflict_spec).copy(deps=False) + ext.external_path = '/fake/path' + ext.concretize() # failure raises exception + def test_regression_issue_4492(self): # Constructing a spec which has no dependencies, but is otherwise # concrete is kind of difficult. What we will do is to concretize