diff options
| author | Todd Gamblin <tgamblin@llnl.gov> | 2020-01-03 23:28:53 -0800 |
|---|---|---|
| committer | Todd Gamblin <tgamblin@llnl.gov> | 2020-03-19 09:00:40 -0700 |
| commit | 7cfa497912d565ae24ae90a8095fde206901a276 (patch) | |
| tree | c5cf79e7f0114d4b534ec15c2f138dac3aa65019 /lib | |
| parent | 0b1b8c3936ba01767b17b20eb54bfacc8376f2c8 (diff) | |
| download | spack-7cfa497912d565ae24ae90a8095fde206901a276.tar.gz spack-7cfa497912d565ae24ae90a8095fde206901a276.tar.bz2 spack-7cfa497912d565ae24ae90a8095fde206901a276.tar.xz spack-7cfa497912d565ae24ae90a8095fde206901a276.zip | |
bugfix: `spack.package.possible_dependencies()` should return virtuals
`spack.package.possible_dependencies()` was forgetting to pass the
`visited` dict to recursive calls; this fixes it
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/spack/spack/package.py | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/lib/spack/spack/package.py b/lib/spack/spack/package.py index 2c8917a9ac..de0f9b3568 100644 --- a/lib/spack/spack/package.py +++ b/lib/spack/spack/package.py @@ -2154,11 +2154,6 @@ def possible_dependencies(*pkg_or_spec, **kwargs): See ``PackageBase.possible_dependencies`` for details. """ - transitive = kwargs.get('transitive', True) - expand_virtuals = kwargs.get('expand_virtuals', True) - deptype = kwargs.get('deptype', 'all') - missing = kwargs.get('missing') - packages = [] for pos in pkg_or_spec: if isinstance(pos, PackageMeta): @@ -2172,8 +2167,7 @@ def possible_dependencies(*pkg_or_spec, **kwargs): visited = {} for pkg in packages: - pkg.possible_dependencies( - transitive, expand_virtuals, deptype, visited, missing) + pkg.possible_dependencies(visited=visited, **kwargs) return visited |