summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorHarmen Stoppels <harmenstoppels@gmail.com>2022-11-02 22:24:59 +0100
committerGitHub <noreply@github.com>2022-11-02 14:24:59 -0700
commitb652fe72d7a55150813c6ffbb68e9c724737fc0a (patch)
tree5ddfcc275079a9049ca790a855736ff17fe63fe3
parent4d5f2e3a37f3b871c0a6c17ac60274a2ce5529f3 (diff)
downloadspack-b652fe72d7a55150813c6ffbb68e9c724737fc0a.tar.gz
spack-b652fe72d7a55150813c6ffbb68e9c724737fc0a.tar.bz2
spack-b652fe72d7a55150813c6ffbb68e9c724737fc0a.tar.xz
spack-b652fe72d7a55150813c6ffbb68e9c724737fc0a.zip
remove `deptype_query` remnants and fix incorrect `deptypes` kwarg (#33670)
* remove deptype_query remnants * deptypes -> deptype These arguments haven't existed since 2017, but `traverse` now fails on unknown **kwargs, so they have finally popped up.
-rw-r--r--lib/spack/spack/build_systems/lua.py2
-rw-r--r--var/spack/repos/builtin/packages/lua/package.py2
-rw-r--r--var/spack/repos/builtin/packages/perl/package.py8
-rw-r--r--var/spack/repos/builtin/packages/r/package.py2
4 files changed, 7 insertions, 7 deletions
diff --git a/lib/spack/spack/build_systems/lua.py b/lib/spack/spack/build_systems/lua.py
index f1c3e43704..722e9153f9 100644
--- a/lib/spack/spack/build_systems/lua.py
+++ b/lib/spack/spack/build_systems/lua.py
@@ -76,7 +76,7 @@ class LuaBuilder(spack.builder.Builder):
def generate_luarocks_config(self, pkg, spec, prefix):
spec = self.pkg.spec
table_entries = []
- for d in spec.traverse(deptypes=("build", "run"), deptype_query="run"):
+ for d in spec.traverse(deptype=("build", "run")):
if d.package.extends(self.pkg.extendee_spec):
table_entries.append(self._generate_tree_line(d.name, d.prefix))
diff --git a/var/spack/repos/builtin/packages/lua/package.py b/var/spack/repos/builtin/packages/lua/package.py
index 4894ab2b91..d04d2036d3 100644
--- a/var/spack/repos/builtin/packages/lua/package.py
+++ b/var/spack/repos/builtin/packages/lua/package.py
@@ -115,7 +115,7 @@ class LuaImplPackage(MakefilePackage):
def _setup_dependent_env_helper(self, env, dependent_spec):
lua_paths = []
- for d in dependent_spec.traverse(deptypes=("build", "run"), deptype_query="run"):
+ for d in dependent_spec.traverse(deptype=("build", "run")):
if d.package.extends(self.spec):
lua_paths.append(os.path.join(d.prefix, self.lua_lib_dir))
lua_paths.append(os.path.join(d.prefix, self.lua_lib64_dir))
diff --git a/var/spack/repos/builtin/packages/perl/package.py b/var/spack/repos/builtin/packages/perl/package.py
index d7b32543fd..b268c86f2b 100644
--- a/var/spack/repos/builtin/packages/perl/package.py
+++ b/var/spack/repos/builtin/packages/perl/package.py
@@ -355,12 +355,12 @@ class Perl(Package): # Perl doesn't use Autotools, it should subclass Package
maker()
maker("install")
- def _setup_dependent_env(self, env, dependent_spec, deptypes):
+ def _setup_dependent_env(self, env, dependent_spec, deptype):
"""Set PATH and PERL5LIB to include the extension and
any other perl extensions it depends on,
assuming they were installed with INSTALL_BASE defined."""
perl_lib_dirs = []
- for d in dependent_spec.traverse(deptype=deptypes):
+ for d in dependent_spec.traverse(deptype=deptype):
if d.package.extends(self.spec):
perl_lib_dirs.append(d.prefix.lib.perl5)
if perl_lib_dirs:
@@ -370,10 +370,10 @@ class Perl(Package): # Perl doesn't use Autotools, it should subclass Package
env.append_path("PATH", self.prefix.bin)
def setup_dependent_build_environment(self, env, dependent_spec):
- self._setup_dependent_env(env, dependent_spec, deptypes=("build", "run", "test"))
+ self._setup_dependent_env(env, dependent_spec, deptype=("build", "run", "test"))
def setup_dependent_run_environment(self, env, dependent_spec):
- self._setup_dependent_env(env, dependent_spec, deptypes=("run",))
+ self._setup_dependent_env(env, dependent_spec, deptype=("run",))
def setup_dependent_package(self, module, dependent_spec):
"""Called before perl modules' install() methods.
diff --git a/var/spack/repos/builtin/packages/r/package.py b/var/spack/repos/builtin/packages/r/package.py
index ffae98f8ef..af5d740b12 100644
--- a/var/spack/repos/builtin/packages/r/package.py
+++ b/var/spack/repos/builtin/packages/r/package.py
@@ -198,7 +198,7 @@ class R(AutotoolsPackage):
# Set R_LIBS to include the library dir for the
# extension and any other R extensions it depends on.
r_libs_path = []
- for d in dependent_spec.traverse(deptype=("build", "run"), deptype_query="run"):
+ for d in dependent_spec.traverse(deptype=("build", "run")):
if d.package.extends(self.spec):
r_libs_path.append(join_path(d.prefix, self.r_lib_dir))