zhangyiss/spack
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

spack audit: account for % new semantic

Browse Source 
Since % means "direct build dependency", we need to exclude
it from the check in audits.
This commit is contained in:
Massimiliano Culpo 2024-11-11 12:24:17 +01:00
parent a8806e494b
commit 8a78eb8c28
No known key found for this signature in database
GPG Key ID: 3E52BB992233066C
1 changed files with 9 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
lib/spack/spack/audit.py
View File

@ -55,6 +55,7 @@ def _search_duplicate_compilers(error_cls):
import spack.builder import spack.builder
import spack.config import spack.config
import spack.deptypes
import spack.fetch_strategy import spack.fetch_strategy
import spack.patch import spack.patch
import spack.repo import spack.repo
@ -1014,7 +1015,14 @@ def _issues_in_depends_on_directive(pkgs, error_cls):
# depends_on('foo+bar ^fee+baz') # depends_on('foo+bar ^fee+baz')
# #
# but we'd like to have two dependencies listed instead. # but we'd like to have two dependencies listed instead.
nested_dependencies = dep.spec.dependencies() nested_dependencies = dep.spec.edges_to_dependencies()
# Filter out pure build dependencies, like:
#
# depends_on('foo+bar%gcc')
#
nested_dependencies = [
x for x in nested_dependencies if x.depflag != spack.deptypes.BUILD
]
if nested_dependencies: if nested_dependencies:
summary = f"{pkg_name}: nested dependency declaration '{dep.spec}'" summary = f"{pkg_name}: nested dependency declaration '{dep.spec}'"
ndir = len(nested_dependencies) + 1 ndir = len(nested_dependencies) + 1