refactor: make PackageBase.possible_dependencies() a class method.
This commit is contained in:
parent
4ac64e6cd8
commit
3f5141d629
@ -525,9 +525,10 @@ def installed_upstream(self):
|
|||||||
|
|
||||||
return self._installed_upstream
|
return self._installed_upstream
|
||||||
|
|
||||||
|
@classmethod
|
||||||
def possible_dependencies(
|
def possible_dependencies(
|
||||||
self, transitive=True, expand_virtuals=True, visited=None):
|
cls, transitive=True, expand_virtuals=True, visited=None):
|
||||||
"""Return set of possible dependencies of this package.
|
"""Return set of possible transitive dependencies of this package.
|
||||||
|
|
||||||
Note: the set returned *includes* the package itself.
|
Note: the set returned *includes* the package itself.
|
||||||
|
|
||||||
@ -539,9 +540,9 @@ def possible_dependencies(
|
|||||||
visited (set): set of names of dependencies visited so far.
|
visited (set): set of names of dependencies visited so far.
|
||||||
"""
|
"""
|
||||||
if visited is None:
|
if visited is None:
|
||||||
visited = set([self.name])
|
visited = set([cls.name])
|
||||||
|
|
||||||
for i, name in enumerate(self.dependencies):
|
for i, name in enumerate(cls.dependencies):
|
||||||
if spack.repo.path.is_virtual(name):
|
if spack.repo.path.is_virtual(name):
|
||||||
if expand_virtuals:
|
if expand_virtuals:
|
||||||
providers = spack.repo.path.providers_for(name)
|
providers = spack.repo.path.providers_for(name)
|
||||||
|
Loading…
Reference in New Issue
Block a user