diff options
| author | Massimiliano Culpo <massimiliano.culpo@gmail.com> | 2020-12-01 10:11:40 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-12-01 10:11:40 +0100 |
| commit | e2033566bf79da12c32cd1807f02b2b9ae4a4e4f (patch) | |
| tree | 98c6c5ed2c9b5352e04a573554b81d61c3318130 /var | |
| parent | 7fd777c3d93d7120c4d2ea3c13c1cd7e083181d2 (diff) | |
| download | spack-e2033566bf79da12c32cd1807f02b2b9ae4a4e4f.tar.gz spack-e2033566bf79da12c32cd1807f02b2b9ae4a4e4f.tar.bz2 spack-e2033566bf79da12c32cd1807f02b2b9ae4a4e4f.tar.xz spack-e2033566bf79da12c32cd1807f02b2b9ae4a4e4f.zip | |
concretizer: remove ad-hoc rule for external packages (#20193)
fixes #20040
Matching compilers among nodes has been prioritized
in #20020. Selection of default variants has been
tuned in #20182. With this setup there is no need
to have an ad-hoc rule for external packages. On
the contrary it should be removed to prefer having
default variant values over more external nodes in
the DAG.
Diffstat (limited to 'var')
| -rw-r--r-- | var/spack/repos/builtin.mock/packages/vdefault-or-external-root/package.py | 15 | ||||
| -rw-r--r-- | var/spack/repos/builtin.mock/packages/vdefault-or-external/package.py | 17 |
2 files changed, 32 insertions, 0 deletions
diff --git a/var/spack/repos/builtin.mock/packages/vdefault-or-external-root/package.py b/var/spack/repos/builtin.mock/packages/vdefault-or-external-root/package.py new file mode 100644 index 0000000000..49011fc458 --- /dev/null +++ b/var/spack/repos/builtin.mock/packages/vdefault-or-external-root/package.py @@ -0,0 +1,15 @@ +# Copyright 2013-2020 Lawrence Livermore National Security, LLC and other +# Spack Project Developers. See the top-level COPYRIGHT file for details. +# +# SPDX-License-Identifier: (Apache-2.0 OR MIT) +class VdefaultOrExternalRoot(Package): + """Test that we don't prefer adding an external to using + a default variant value. + """ + + homepage = 'https://www.example.org' + url = 'https://example.org/files/v3.4/cmake-3.4.3.tar.gz' + + version('1.0', '4cb3ff35b2472aae70f542116d616e63') + + depends_on('vdefault-or-external') diff --git a/var/spack/repos/builtin.mock/packages/vdefault-or-external/package.py b/var/spack/repos/builtin.mock/packages/vdefault-or-external/package.py new file mode 100644 index 0000000000..80c674917e --- /dev/null +++ b/var/spack/repos/builtin.mock/packages/vdefault-or-external/package.py @@ -0,0 +1,17 @@ +# Copyright 2013-2020 Lawrence Livermore National Security, LLC and other +# Spack Project Developers. See the top-level COPYRIGHT file for details. +# +# SPDX-License-Identifier: (Apache-2.0 OR MIT) +class VdefaultOrExternal(Package): + """Test that we don't prefer adding an external to using + a default variant value. + """ + + homepage = 'https://www.example.org' + url = 'https://example.org/files/v3.4/cmake-3.4.3.tar.gz' + + version('1.0', '4cb3ff35b2472aae70f542116d616e63') + + variant('external', default=False, description='nope') + + depends_on('externaltool', when='+external') |