summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--lib/spack/spack/package.py2
-rw-r--r--lib/spack/spack/util/executable.py12
-rw-r--r--var/spack/repos/builtin/packages/acfl/package.py1
-rw-r--r--var/spack/repos/builtin/packages/aqlprofile/package.py1
-rw-r--r--var/spack/repos/builtin/packages/armpl-gcc/package.py2
-rw-r--r--var/spack/repos/builtin/packages/axom/package.py1
-rw-r--r--var/spack/repos/builtin/packages/claw/package.py2
-rw-r--r--var/spack/repos/builtin/packages/codipack/package.py1
-rw-r--r--var/spack/repos/builtin/packages/coin3d/package.py2
-rw-r--r--var/spack/repos/builtin/packages/cp2k/package.py2
-rw-r--r--var/spack/repos/builtin/packages/cxx/package.py2
-rw-r--r--var/spack/repos/builtin/packages/dbus/package.py2
-rw-r--r--var/spack/repos/builtin/packages/eccodes/package.py2
-rw-r--r--var/spack/repos/builtin/packages/elsi/package.py1
-rw-r--r--var/spack/repos/builtin/packages/esmf/package.py3
-rw-r--r--var/spack/repos/builtin/packages/flux-core/package.py3
-rw-r--r--var/spack/repos/builtin/packages/flux-sched/package.py3
-rw-r--r--var/spack/repos/builtin/packages/flux-security/package.py3
-rw-r--r--var/spack/repos/builtin/packages/fsl/package.py1
-rw-r--r--var/spack/repos/builtin/packages/gasnet/package.py3
-rw-r--r--var/spack/repos/builtin/packages/gaussian-view/package.py1
-rw-r--r--var/spack/repos/builtin/packages/gcc/package.py2
-rw-r--r--var/spack/repos/builtin/packages/gem5/package.py1
-rw-r--r--var/spack/repos/builtin/packages/git/package.py1
-rw-r--r--var/spack/repos/builtin/packages/glib/package.py2
-rw-r--r--var/spack/repos/builtin/packages/gromacs/package.py1
-rw-r--r--var/spack/repos/builtin/packages/gtkplus/package.py2
-rw-r--r--var/spack/repos/builtin/packages/hdf/package.py2
-rw-r--r--var/spack/repos/builtin/packages/hip/package.py1
-rw-r--r--var/spack/repos/builtin/packages/hipblas/package.py1
-rw-r--r--var/spack/repos/builtin/packages/hipcub/package.py1
-rw-r--r--var/spack/repos/builtin/packages/hipfft/package.py1
-rw-r--r--var/spack/repos/builtin/packages/hiprand/package.py1
-rw-r--r--var/spack/repos/builtin/packages/hipsolver/package.py1
-rw-r--r--var/spack/repos/builtin/packages/hipsparse/package.py1
-rw-r--r--var/spack/repos/builtin/packages/hipsparselt/package.py1
-rw-r--r--var/spack/repos/builtin/packages/hiredis/package.py2
-rw-r--r--var/spack/repos/builtin/packages/hpctoolkit/package.py2
-rw-r--r--var/spack/repos/builtin/packages/icu4c/package.py2
-rw-r--r--var/spack/repos/builtin/packages/intel-oneapi-mkl/package.py2
-rw-r--r--var/spack/repos/builtin/packages/intel/package.py3
-rw-r--r--var/spack/repos/builtin/packages/ipm/package.py1
-rw-r--r--var/spack/repos/builtin/packages/jsoncpp/package.py2
-rw-r--r--var/spack/repos/builtin/packages/kokkos/package.py1
-rw-r--r--var/spack/repos/builtin/packages/lcms/package.py1
-rw-r--r--var/spack/repos/builtin/packages/libaec/package.py2
-rw-r--r--var/spack/repos/builtin/packages/libdrm/package.py2
-rw-r--r--var/spack/repos/builtin/packages/libdwarf/package.py2
-rw-r--r--var/spack/repos/builtin/packages/libepoxy/package.py2
-rw-r--r--var/spack/repos/builtin/packages/libjpeg-turbo/package.py1
-rw-r--r--var/spack/repos/builtin/packages/libssh2/package.py2
-rw-r--r--var/spack/repos/builtin/packages/libszip/package.py2
-rw-r--r--var/spack/repos/builtin/packages/libuv/package.py1
-rw-r--r--var/spack/repos/builtin/packages/likwid/package.py1
-rw-r--r--var/spack/repos/builtin/packages/llvm-doe/package.py3
-rw-r--r--var/spack/repos/builtin/packages/llvm/package.py4
-rw-r--r--var/spack/repos/builtin/packages/lua/package.py1
-rw-r--r--var/spack/repos/builtin/packages/magma/package.py1
-rw-r--r--var/spack/repos/builtin/packages/mapl/package.py1
-rw-r--r--var/spack/repos/builtin/packages/mesa/package.py1
-rw-r--r--var/spack/repos/builtin/packages/mmg/package.py1
-rw-r--r--var/spack/repos/builtin/packages/mpas-model/package.py1
-rw-r--r--var/spack/repos/builtin/packages/mpich/package.py1
-rw-r--r--var/spack/repos/builtin/packages/msvc/package.py1
-rw-r--r--var/spack/repos/builtin/packages/mvapich2/package.py1
-rw-r--r--var/spack/repos/builtin/packages/nasm/package.py1
-rw-r--r--var/spack/repos/builtin/packages/netcdf-cxx4/package.py2
-rw-r--r--var/spack/repos/builtin/packages/netcdf-fortran/package.py2
-rw-r--r--var/spack/repos/builtin/packages/ninja-fortran/package.py2
-rw-r--r--var/spack/repos/builtin/packages/ninja/package.py1
-rw-r--r--var/spack/repos/builtin/packages/openmpi/package.py2
-rw-r--r--var/spack/repos/builtin/packages/papi/package.py1
-rw-r--r--var/spack/repos/builtin/packages/parallel-netcdf/package.py2
-rw-r--r--var/spack/repos/builtin/packages/pcre2/package.py2
-rw-r--r--var/spack/repos/builtin/packages/perl/package.py4
-rw-r--r--var/spack/repos/builtin/packages/phist/package.py2
-rw-r--r--var/spack/repos/builtin/packages/pixman/package.py2
-rw-r--r--var/spack/repos/builtin/packages/py-matplotlib/package.py2
-rw-r--r--var/spack/repos/builtin/packages/py-pennylane-lightning-kokkos/package.py1
-rw-r--r--var/spack/repos/builtin/packages/py-pennylane-lightning/package.py1
-rw-r--r--var/spack/repos/builtin/packages/python/package.py5
-rw-r--r--var/spack/repos/builtin/packages/root/package.py1
-rw-r--r--var/spack/repos/builtin/packages/rpcsvc-proto/package.py1
-rw-r--r--var/spack/repos/builtin/packages/sccache/package.py1
-rw-r--r--var/spack/repos/builtin/packages/seqkit/package.py1
-rw-r--r--var/spack/repos/builtin/packages/serialbox/package.py2
-rw-r--r--var/spack/repos/builtin/packages/sherpa/package.py2
-rw-r--r--var/spack/repos/builtin/packages/singularity-eos/package.py1
-rw-r--r--var/spack/repos/builtin/packages/singularityce/package.py1
-rw-r--r--var/spack/repos/builtin/packages/spectrum-mpi/package.py1
-rw-r--r--var/spack/repos/builtin/packages/sqlite/package.py2
-rw-r--r--var/spack/repos/builtin/packages/strumpack/package.py1
-rw-r--r--var/spack/repos/builtin/packages/szx/package.py2
-rw-r--r--var/spack/repos/builtin/packages/tcl/package.py2
-rw-r--r--var/spack/repos/builtin/packages/thrust/package.py2
-rw-r--r--var/spack/repos/builtin/packages/trilinos/package.py1
-rw-r--r--var/spack/repos/builtin/packages/upcxx/package.py1
-rw-r--r--var/spack/repos/builtin/packages/util-macros/package.py1
-rw-r--r--var/spack/repos/builtin/packages/wayland-protocols/package.py1
-rw-r--r--var/spack/repos/builtin/packages/yafyaml/package.py1
-rw-r--r--var/spack/repos/builtin/packages/zziplib/package.py1
101 files changed, 125 insertions, 49 deletions
diff --git a/lib/spack/spack/package.py b/lib/spack/spack/package.py
index 8a7795b2ce..525721ebb3 100644
--- a/lib/spack/spack/package.py
+++ b/lib/spack/spack/package.py
@@ -11,7 +11,7 @@ Everything in this module is automatically imported into Spack package files.
from os import chdir, environ, getcwd, makedirs, mkdir, remove, removedirs
from shutil import move, rmtree
-from spack.error import InstallError
+from spack.error import InstallError, NoHeadersError, NoLibrariesError
# Emulate some shell commands for convenience
env = environ
diff --git a/lib/spack/spack/util/executable.py b/lib/spack/spack/util/executable.py
index 0c1901cb1a..83534f8000 100644
--- a/lib/spack/spack/util/executable.py
+++ b/lib/spack/spack/util/executable.py
@@ -12,9 +12,9 @@ from pathlib import Path, PurePath
import llnl.util.tty as tty
import spack.error
-from spack.util.environment import EnvironmentModifications
+import spack.util.environment
-__all__ = ["Executable", "which", "ProcessError"]
+__all__ = ["Executable", "which", "which_string", "ProcessError"]
class Executable:
@@ -29,7 +29,7 @@ class Executable:
self.default_env = {}
- self.default_envmod = EnvironmentModifications()
+ self.default_envmod = spack.util.environment.EnvironmentModifications()
self.returncode = None
self.ignore_quotes = False
@@ -168,17 +168,15 @@ class Executable:
self.default_envmod.apply_modifications(env)
env.update(self.default_env)
- from spack.util.environment import EnvironmentModifications # no cycle
-
# Apply env argument
- if isinstance(env_arg, EnvironmentModifications):
+ if isinstance(env_arg, spack.util.environment.EnvironmentModifications):
env_arg.apply_modifications(env)
elif env_arg:
env.update(env_arg)
# Apply extra env
extra_env = kwargs.get("extra_env", {})
- if isinstance(extra_env, EnvironmentModifications):
+ if isinstance(extra_env, spack.util.environment.EnvironmentModifications):
extra_env.apply_modifications(env)
else:
env.update(extra_env)
diff --git a/var/spack/repos/builtin/packages/acfl/package.py b/var/spack/repos/builtin/packages/acfl/package.py
index 4fa2f553e8..6e1d9fa618 100644
--- a/var/spack/repos/builtin/packages/acfl/package.py
+++ b/var/spack/repos/builtin/packages/acfl/package.py
@@ -4,6 +4,7 @@
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
import os
+import spack.platforms
from spack.package import *
_os_map_before_23 = {
diff --git a/var/spack/repos/builtin/packages/aqlprofile/package.py b/var/spack/repos/builtin/packages/aqlprofile/package.py
index 199ed72d0f..ca5a74b211 100644
--- a/var/spack/repos/builtin/packages/aqlprofile/package.py
+++ b/var/spack/repos/builtin/packages/aqlprofile/package.py
@@ -5,6 +5,7 @@
import os
+import spack.platforms
from spack.package import *
_versions = {
diff --git a/var/spack/repos/builtin/packages/armpl-gcc/package.py b/var/spack/repos/builtin/packages/armpl-gcc/package.py
index 25f187781e..3da17ef3e4 100644
--- a/var/spack/repos/builtin/packages/armpl-gcc/package.py
+++ b/var/spack/repos/builtin/packages/armpl-gcc/package.py
@@ -5,6 +5,8 @@
import os
+import spack.error
+import spack.platforms
from spack.package import *
_os_map_before_23 = {
diff --git a/var/spack/repos/builtin/packages/axom/package.py b/var/spack/repos/builtin/packages/axom/package.py
index 5791fa3df9..23e29fa32f 100644
--- a/var/spack/repos/builtin/packages/axom/package.py
+++ b/var/spack/repos/builtin/packages/axom/package.py
@@ -9,7 +9,6 @@ import socket
from os.path import join as pjoin
from spack.package import *
-from spack.util.executable import which_string
def get_spec_path(spec, package_name, path_replacements={}, use_bin=False):
diff --git a/var/spack/repos/builtin/packages/claw/package.py b/var/spack/repos/builtin/packages/claw/package.py
index 913b1b82f6..481745cb90 100644
--- a/var/spack/repos/builtin/packages/claw/package.py
+++ b/var/spack/repos/builtin/packages/claw/package.py
@@ -5,6 +5,8 @@
import os
+import spack.compilers
+import spack.spec
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/codipack/package.py b/var/spack/repos/builtin/packages/codipack/package.py
index 2347df15ce..4c2fe3886b 100644
--- a/var/spack/repos/builtin/packages/codipack/package.py
+++ b/var/spack/repos/builtin/packages/codipack/package.py
@@ -3,6 +3,7 @@
#
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
+import spack.build_systems.generic
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/coin3d/package.py b/var/spack/repos/builtin/packages/coin3d/package.py
index 6cffab95cb..571c843cb0 100644
--- a/var/spack/repos/builtin/packages/coin3d/package.py
+++ b/var/spack/repos/builtin/packages/coin3d/package.py
@@ -3,6 +3,8 @@
#
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
+import spack.build_systems.autotools
+import spack.build_systems.cmake
from spack.package import *
from spack.pkg.builtin.boost import Boost
diff --git a/var/spack/repos/builtin/packages/cp2k/package.py b/var/spack/repos/builtin/packages/cp2k/package.py
index dc24d525f3..5a478ea985 100644
--- a/var/spack/repos/builtin/packages/cp2k/package.py
+++ b/var/spack/repos/builtin/packages/cp2k/package.py
@@ -7,9 +7,7 @@ import os
import os.path
import sys
-import spack.platforms
import spack.util.environment
-import spack.util.executable
from spack.build_environment import dso_suffix
from spack.build_systems import cmake, makefile
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/cxx/package.py b/var/spack/repos/builtin/packages/cxx/package.py
index bedd235fd8..ac5c5cc5a6 100644
--- a/var/spack/repos/builtin/packages/cxx/package.py
+++ b/var/spack/repos/builtin/packages/cxx/package.py
@@ -5,6 +5,8 @@
import os
+import spack.compilers
+import spack.spec
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/dbus/package.py b/var/spack/repos/builtin/packages/dbus/package.py
index 4b54a5fce8..3f66773377 100644
--- a/var/spack/repos/builtin/packages/dbus/package.py
+++ b/var/spack/repos/builtin/packages/dbus/package.py
@@ -3,6 +3,8 @@
#
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
+import spack.build_systems.autotools
+import spack.build_systems.meson
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/eccodes/package.py b/var/spack/repos/builtin/packages/eccodes/package.py
index fc8f89f71a..3c1e955cb2 100644
--- a/var/spack/repos/builtin/packages/eccodes/package.py
+++ b/var/spack/repos/builtin/packages/eccodes/package.py
@@ -302,7 +302,7 @@ class Eccodes(CMakePackage):
return libs
msg = "Unable to recursively locate {0} {1} libraries in {2}"
- raise spack.error.NoLibrariesError(
+ raise NoLibrariesError(
msg.format("shared" if shared else "static", self.spec.name, self.spec.prefix)
)
diff --git a/var/spack/repos/builtin/packages/elsi/package.py b/var/spack/repos/builtin/packages/elsi/package.py
index e06936a754..43573255a0 100644
--- a/var/spack/repos/builtin/packages/elsi/package.py
+++ b/var/spack/repos/builtin/packages/elsi/package.py
@@ -4,7 +4,6 @@
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
import os.path
-from spack.error import NoHeadersError
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/esmf/package.py b/var/spack/repos/builtin/packages/esmf/package.py
index 9eb145478a..20a7223e63 100644
--- a/var/spack/repos/builtin/packages/esmf/package.py
+++ b/var/spack/repos/builtin/packages/esmf/package.py
@@ -6,6 +6,9 @@
import os
import sys
+import spack.build_systems.makefile
+import spack.build_systems.python
+import spack.compiler
from spack.build_environment import dso_suffix, stat_suffix
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/flux-core/package.py b/var/spack/repos/builtin/packages/flux-core/package.py
index 52eddda9fc..71d2109789 100644
--- a/var/spack/repos/builtin/packages/flux-core/package.py
+++ b/var/spack/repos/builtin/packages/flux-core/package.py
@@ -5,7 +5,6 @@
import os
-import spack.util.executable
from spack.package import *
@@ -156,7 +155,7 @@ class FluxCore(AutotoolsPackage):
git("fetch", "--unshallow")
git("config", "remote.origin.fetch", "+refs/heads/*:refs/remotes/origin/*")
git("fetch", "origin")
- except spack.util.executable.ProcessError:
+ except ProcessError:
git("fetch")
def autoreconf(self, spec, prefix):
diff --git a/var/spack/repos/builtin/packages/flux-sched/package.py b/var/spack/repos/builtin/packages/flux-sched/package.py
index 66d5532bf3..e9fa209c31 100644
--- a/var/spack/repos/builtin/packages/flux-sched/package.py
+++ b/var/spack/repos/builtin/packages/flux-sched/package.py
@@ -5,7 +5,6 @@
import os
-import spack.util.executable
from spack.build_systems.autotools import AutotoolsBuilder
from spack.build_systems.cmake import CMakeBuilder
from spack.package import *
@@ -139,7 +138,7 @@ class FluxSched(CMakePackage, AutotoolsPackage):
git("fetch", "--unshallow")
git("config", "remote.origin.fetch", "+refs/heads/*:refs/remotes/origin/*")
git("fetch", "origin")
- except spack.util.executable.ProcessError:
+ except ProcessError:
git("fetch")
def autoreconf(self, spec, prefix):
diff --git a/var/spack/repos/builtin/packages/flux-security/package.py b/var/spack/repos/builtin/packages/flux-security/package.py
index 23183c6d3b..6431578476 100644
--- a/var/spack/repos/builtin/packages/flux-security/package.py
+++ b/var/spack/repos/builtin/packages/flux-security/package.py
@@ -5,7 +5,6 @@
import os
-import spack.util.executable
from spack.package import *
@@ -54,7 +53,7 @@ class FluxSecurity(AutotoolsPackage):
git("fetch", "--unshallow")
git("config", "remote.origin.fetch", "+refs/heads/*:refs/remotes/origin/*")
git("fetch", "origin")
- except spack.util.executable.ProcessError:
+ except ProcessError:
git("fetch")
def autoreconf(self, spec, prefix):
diff --git a/var/spack/repos/builtin/packages/fsl/package.py b/var/spack/repos/builtin/packages/fsl/package.py
index 690911373e..746005e3d1 100644
--- a/var/spack/repos/builtin/packages/fsl/package.py
+++ b/var/spack/repos/builtin/packages/fsl/package.py
@@ -6,6 +6,7 @@
import glob
import os
+import spack.util.environment
from spack.package import *
from spack.util.environment import EnvironmentModifications
diff --git a/var/spack/repos/builtin/packages/gasnet/package.py b/var/spack/repos/builtin/packages/gasnet/package.py
index a610079712..a929e6b957 100644
--- a/var/spack/repos/builtin/packages/gasnet/package.py
+++ b/var/spack/repos/builtin/packages/gasnet/package.py
@@ -5,6 +5,7 @@
import os
+import spack.main
from spack.package import *
@@ -145,7 +146,7 @@ class Gasnet(Package, CudaPackage, ROCmPackage):
try:
git = which("git")
git("describe", "--long", "--always", output="version.git")
- except spack.util.executable.ProcessError:
+ except ProcessError:
spack.main.send_warning_to_tty("Omitting version stamp due to git error")
# The GASNet-EX library has a highly multi-dimensional configure space,
diff --git a/var/spack/repos/builtin/packages/gaussian-view/package.py b/var/spack/repos/builtin/packages/gaussian-view/package.py
index eec0877b57..f0b241aed8 100644
--- a/var/spack/repos/builtin/packages/gaussian-view/package.py
+++ b/var/spack/repos/builtin/packages/gaussian-view/package.py
@@ -8,6 +8,7 @@ import os
import llnl.util.tty as tty
+import spack.tengine
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/gcc/package.py b/var/spack/repos/builtin/packages/gcc/package.py
index 60b8d44b7a..abf1529159 100644
--- a/var/spack/repos/builtin/packages/gcc/package.py
+++ b/var/spack/repos/builtin/packages/gcc/package.py
@@ -11,8 +11,8 @@ import archspec.cpu
import llnl.util.tty as tty
from llnl.util.symlink import readlink
+import spack.compiler
import spack.platforms
-import spack.util.executable
import spack.util.libc
from spack.operating_systems.mac_os import macos_sdk_path, macos_version
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/gem5/package.py b/var/spack/repos/builtin/packages/gem5/package.py
index f18a478c3b..194d2743b7 100644
--- a/var/spack/repos/builtin/packages/gem5/package.py
+++ b/var/spack/repos/builtin/packages/gem5/package.py
@@ -4,6 +4,7 @@
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
+import spack.config
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/git/package.py b/var/spack/repos/builtin/packages/git/package.py
index 713fd0abf8..9b5abba5fc 100644
--- a/var/spack/repos/builtin/packages/git/package.py
+++ b/var/spack/repos/builtin/packages/git/package.py
@@ -6,6 +6,7 @@
import os
import re
+import spack.fetch_strategy
from spack.package import *
from spack.util.environment import is_system_path
diff --git a/var/spack/repos/builtin/packages/glib/package.py b/var/spack/repos/builtin/packages/glib/package.py
index 057d180dd6..982ea5449e 100644
--- a/var/spack/repos/builtin/packages/glib/package.py
+++ b/var/spack/repos/builtin/packages/glib/package.py
@@ -5,6 +5,8 @@
import os.path
+import spack.build_systems.autotools
+import spack.build_systems.meson
from spack.package import *
from spack.util.environment import is_system_path
diff --git a/var/spack/repos/builtin/packages/gromacs/package.py b/var/spack/repos/builtin/packages/gromacs/package.py
index 5703587a65..055cbebe14 100644
--- a/var/spack/repos/builtin/packages/gromacs/package.py
+++ b/var/spack/repos/builtin/packages/gromacs/package.py
@@ -7,6 +7,7 @@ import os
import llnl.util.filesystem as fs
+import spack.build_systems.cmake
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/gtkplus/package.py b/var/spack/repos/builtin/packages/gtkplus/package.py
index 158d7adfd7..68b21046fe 100644
--- a/var/spack/repos/builtin/packages/gtkplus/package.py
+++ b/var/spack/repos/builtin/packages/gtkplus/package.py
@@ -3,6 +3,8 @@
#
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
+import spack.build_systems.autotools
+import spack.build_systems.meson
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/hdf/package.py b/var/spack/repos/builtin/packages/hdf/package.py
index 85acc3dbb3..9531524c0c 100644
--- a/var/spack/repos/builtin/packages/hdf/package.py
+++ b/var/spack/repos/builtin/packages/hdf/package.py
@@ -127,7 +127,7 @@ class Hdf(AutotoolsPackage):
if not libs:
msg = "Unable to recursively locate {0} {1} libraries in {2}"
- raise spack.error.NoLibrariesError(
+ raise NoLibrariesError(
msg.format("shared" if shared else "static", self.spec.name, self.spec.prefix)
)
diff --git a/var/spack/repos/builtin/packages/hip/package.py b/var/spack/repos/builtin/packages/hip/package.py
index 91dc94d33c..fa2c892f16 100644
--- a/var/spack/repos/builtin/packages/hip/package.py
+++ b/var/spack/repos/builtin/packages/hip/package.py
@@ -6,6 +6,7 @@
import os
import re
+import spack.build_environment
from spack.hooks.sbang import filter_shebang
from spack.package import *
from spack.util.prefix import Prefix
diff --git a/var/spack/repos/builtin/packages/hipblas/package.py b/var/spack/repos/builtin/packages/hipblas/package.py
index e4b0ccc8d5..91bbe38c62 100644
--- a/var/spack/repos/builtin/packages/hipblas/package.py
+++ b/var/spack/repos/builtin/packages/hipblas/package.py
@@ -5,6 +5,7 @@
import re
+import spack.variant
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/hipcub/package.py b/var/spack/repos/builtin/packages/hipcub/package.py
index 5a85b8eb58..06030e38a1 100644
--- a/var/spack/repos/builtin/packages/hipcub/package.py
+++ b/var/spack/repos/builtin/packages/hipcub/package.py
@@ -3,6 +3,7 @@
#
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
+import spack.variant
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/hipfft/package.py b/var/spack/repos/builtin/packages/hipfft/package.py
index e70b311203..ae07ec2670 100644
--- a/var/spack/repos/builtin/packages/hipfft/package.py
+++ b/var/spack/repos/builtin/packages/hipfft/package.py
@@ -4,6 +4,7 @@
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
+import spack.variant
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/hiprand/package.py b/var/spack/repos/builtin/packages/hiprand/package.py
index 61b1702680..d78282fcfc 100644
--- a/var/spack/repos/builtin/packages/hiprand/package.py
+++ b/var/spack/repos/builtin/packages/hiprand/package.py
@@ -5,6 +5,7 @@
import re
+import spack.variant
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/hipsolver/package.py b/var/spack/repos/builtin/packages/hipsolver/package.py
index 5df642cd90..4470f78226 100644
--- a/var/spack/repos/builtin/packages/hipsolver/package.py
+++ b/var/spack/repos/builtin/packages/hipsolver/package.py
@@ -6,6 +6,7 @@
import os
import re
+import spack.variant
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/hipsparse/package.py b/var/spack/repos/builtin/packages/hipsparse/package.py
index 65a9a9a18f..0aa54869df 100644
--- a/var/spack/repos/builtin/packages/hipsparse/package.py
+++ b/var/spack/repos/builtin/packages/hipsparse/package.py
@@ -5,6 +5,7 @@
import re
+import spack.variant
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/hipsparselt/package.py b/var/spack/repos/builtin/packages/hipsparselt/package.py
index e96305ba2c..4c882cc264 100644
--- a/var/spack/repos/builtin/packages/hipsparselt/package.py
+++ b/var/spack/repos/builtin/packages/hipsparselt/package.py
@@ -4,6 +4,7 @@
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
+import spack.variant
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/hiredis/package.py b/var/spack/repos/builtin/packages/hiredis/package.py
index 39eb4aaa5b..9d0a5ba955 100644
--- a/var/spack/repos/builtin/packages/hiredis/package.py
+++ b/var/spack/repos/builtin/packages/hiredis/package.py
@@ -3,6 +3,8 @@
#
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
+import spack.build_systems.cmake
+import spack.build_systems.makefile
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/hpctoolkit/package.py b/var/spack/repos/builtin/packages/hpctoolkit/package.py
index 548bfad92c..33d0768e3d 100644
--- a/var/spack/repos/builtin/packages/hpctoolkit/package.py
+++ b/var/spack/repos/builtin/packages/hpctoolkit/package.py
@@ -9,6 +9,8 @@ import tempfile
import llnl.util.tty as tty
+import spack.build_systems.autotools
+import spack.build_systems.meson
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/icu4c/package.py b/var/spack/repos/builtin/packages/icu4c/package.py
index 97f89cfc29..10189b1b85 100644
--- a/var/spack/repos/builtin/packages/icu4c/package.py
+++ b/var/spack/repos/builtin/packages/icu4c/package.py
@@ -5,6 +5,8 @@
import pathlib
+import spack.build_systems.autotools
+import spack.build_systems.msbuild
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/intel-oneapi-mkl/package.py b/var/spack/repos/builtin/packages/intel-oneapi-mkl/package.py
index c233490408..04b916d211 100644
--- a/var/spack/repos/builtin/packages/intel-oneapi-mkl/package.py
+++ b/var/spack/repos/builtin/packages/intel-oneapi-mkl/package.py
@@ -284,7 +284,7 @@ class IntelOneapiMkl(IntelOneApiLibraryPackage):
try:
if self.spec.satisfies("+cluster ^mpi"):
resolved_libs = resolved_libs + self.spec["mpi"].libs
- except spack.error.NoLibrariesError:
+ except NoLibrariesError:
pass
if self.spec.satisfies("threads=openmp"):
diff --git a/var/spack/repos/builtin/packages/intel/package.py b/var/spack/repos/builtin/packages/intel/package.py
index bdf53db9a8..51bae70edb 100644
--- a/var/spack/repos/builtin/packages/intel/package.py
+++ b/var/spack/repos/builtin/packages/intel/package.py
@@ -6,6 +6,7 @@ import re
import llnl.util.tty as tty
+import spack.compiler
from spack.package import *
@@ -240,7 +241,7 @@ class Intel(IntelPackage):
match = version_regex.search(output)
if match:
return match.group(1)
- except spack.util.executable.ProcessError:
+ except ProcessError:
pass
except Exception as e:
tty.debug(str(e))
diff --git a/var/spack/repos/builtin/packages/ipm/package.py b/var/spack/repos/builtin/packages/ipm/package.py
index 9eb248ff52..cf0c732280 100644
--- a/var/spack/repos/builtin/packages/ipm/package.py
+++ b/var/spack/repos/builtin/packages/ipm/package.py
@@ -4,7 +4,6 @@
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
from spack.package import *
-from spack.util.executable import Executable
class Ipm(AutotoolsPackage):
diff --git a/var/spack/repos/builtin/packages/jsoncpp/package.py b/var/spack/repos/builtin/packages/jsoncpp/package.py
index 747aa8b205..c176626483 100644
--- a/var/spack/repos/builtin/packages/jsoncpp/package.py
+++ b/var/spack/repos/builtin/packages/jsoncpp/package.py
@@ -3,6 +3,8 @@
#
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
+import spack.build_systems.cmake
+import spack.build_systems.meson
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/kokkos/package.py b/var/spack/repos/builtin/packages/kokkos/package.py
index 7eeac574d9..ae8c0a9d6c 100644
--- a/var/spack/repos/builtin/packages/kokkos/package.py
+++ b/var/spack/repos/builtin/packages/kokkos/package.py
@@ -6,6 +6,7 @@ import os.path
import llnl.util.lang as lang
+import spack.build_systems.cmake
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/lcms/package.py b/var/spack/repos/builtin/packages/lcms/package.py
index ce452131b2..dbfe54508a 100644
--- a/var/spack/repos/builtin/packages/lcms/package.py
+++ b/var/spack/repos/builtin/packages/lcms/package.py
@@ -5,6 +5,7 @@
import pathlib
+import spack.build_systems.msbuild
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/libaec/package.py b/var/spack/repos/builtin/packages/libaec/package.py
index f0498b3fd0..12a059907e 100644
--- a/var/spack/repos/builtin/packages/libaec/package.py
+++ b/var/spack/repos/builtin/packages/libaec/package.py
@@ -48,7 +48,7 @@ class Libaec(CMakePackage):
if not libs:
msg = "Unable to recursively locate {0} {1} libraries in {2}"
- raise spack.error.NoLibrariesError(
+ raise NoLibrariesError(
msg.format("shared" if shared else "static", self.spec.name, self.spec.prefix)
)
return libs
diff --git a/var/spack/repos/builtin/packages/libdrm/package.py b/var/spack/repos/builtin/packages/libdrm/package.py
index 5a6aa91b27..6c68e5248f 100644
--- a/var/spack/repos/builtin/packages/libdrm/package.py
+++ b/var/spack/repos/builtin/packages/libdrm/package.py
@@ -3,6 +3,8 @@
#
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
+import spack.build_systems.autotools
+import spack.build_systems.meson
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/libdwarf/package.py b/var/spack/repos/builtin/packages/libdwarf/package.py
index e62211790b..89b6e87327 100644
--- a/var/spack/repos/builtin/packages/libdwarf/package.py
+++ b/var/spack/repos/builtin/packages/libdwarf/package.py
@@ -6,6 +6,8 @@
import os
import sys
+import spack.build_systems.cmake
+import spack.build_systems.generic
from spack.package import *
# Only build certain parts of dwarf because the other ones break.
diff --git a/var/spack/repos/builtin/packages/libepoxy/package.py b/var/spack/repos/builtin/packages/libepoxy/package.py
index 800dbc7303..c991fdb371 100644
--- a/var/spack/repos/builtin/packages/libepoxy/package.py
+++ b/var/spack/repos/builtin/packages/libepoxy/package.py
@@ -3,6 +3,8 @@
#
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
+import spack.build_systems.autotools
+import spack.build_systems.meson
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/libjpeg-turbo/package.py b/var/spack/repos/builtin/packages/libjpeg-turbo/package.py
index 65b07ad458..d88e2984ef 100644
--- a/var/spack/repos/builtin/packages/libjpeg-turbo/package.py
+++ b/var/spack/repos/builtin/packages/libjpeg-turbo/package.py
@@ -5,6 +5,7 @@
import sys
+import spack.build_systems.cmake
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/libssh2/package.py b/var/spack/repos/builtin/packages/libssh2/package.py
index 754d593c4b..8b1b425a71 100644
--- a/var/spack/repos/builtin/packages/libssh2/package.py
+++ b/var/spack/repos/builtin/packages/libssh2/package.py
@@ -3,6 +3,8 @@
#
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
+import spack.build_systems.autotools
+import spack.build_systems.cmake
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/libszip/package.py b/var/spack/repos/builtin/packages/libszip/package.py
index 107c821272..2314879927 100644
--- a/var/spack/repos/builtin/packages/libszip/package.py
+++ b/var/spack/repos/builtin/packages/libszip/package.py
@@ -34,7 +34,7 @@ class Libszip(AutotoolsPackage):
if not libs:
msg = "Unable to recursively locate {0} {1} libraries in {2}"
- raise spack.error.NoLibrariesError(
+ raise NoLibrariesError(
msg.format("shared" if shared else "static", self.spec.name, self.spec.prefix)
)
return libs
diff --git a/var/spack/repos/builtin/packages/libuv/package.py b/var/spack/repos/builtin/packages/libuv/package.py
index 5b578d419e..69cce8f9aa 100644
--- a/var/spack/repos/builtin/packages/libuv/package.py
+++ b/var/spack/repos/builtin/packages/libuv/package.py
@@ -5,6 +5,7 @@
import sys
import spack.build_systems
+import spack.build_systems.autotools
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/likwid/package.py b/var/spack/repos/builtin/packages/likwid/package.py
index be7990506f..65b72202d2 100644
--- a/var/spack/repos/builtin/packages/likwid/package.py
+++ b/var/spack/repos/builtin/packages/likwid/package.py
@@ -8,6 +8,7 @@ import os
import llnl.util.tty as tty
+import spack.tengine
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/llvm-doe/package.py b/var/spack/repos/builtin/packages/llvm-doe/package.py
index 6bf4f67e50..8aae32d5ac 100644
--- a/var/spack/repos/builtin/packages/llvm-doe/package.py
+++ b/var/spack/repos/builtin/packages/llvm-doe/package.py
@@ -9,7 +9,6 @@ import sys
import llnl.util.tty as tty
-import spack.util.executable
from spack.build_systems.cmake import get_cmake_prefix_path
from spack.package import *
@@ -258,7 +257,7 @@ class LlvmDoe(CMakePackage, CudaPackage):
match = version_regex.search(output)
if match:
return match.group(match.lastindex)
- except spack.util.executable.ProcessError:
+ except ProcessError:
pass
except Exception as e:
tty.debug(e)
diff --git a/var/spack/repos/builtin/packages/llvm/package.py b/var/spack/repos/builtin/packages/llvm/package.py
index 0dc66b9503..81ee8b52c0 100644
--- a/var/spack/repos/builtin/packages/llvm/package.py
+++ b/var/spack/repos/builtin/packages/llvm/package.py
@@ -10,7 +10,7 @@ import sys
import llnl.util.tty as tty
from llnl.util.lang import classproperty
-import spack.util.executable
+import spack.compilers
from spack.build_systems.cmake import get_cmake_prefix_path
from spack.package import *
from spack.package_base import PackageBase
@@ -687,7 +687,7 @@ class Llvm(CMakePackage, CudaPackage, LlvmDetection, CompilerPackage):
match = re.search(cls.compiler_version_regex, output)
if match:
return match.group(match.lastindex)
- except spack.util.executable.ProcessError:
+ except ProcessError:
pass
except Exception as e:
tty.debug(e)
diff --git a/var/spack/repos/builtin/packages/lua/package.py b/var/spack/repos/builtin/packages/lua/package.py
index 8f791b5cd8..5a12e58e44 100644
--- a/var/spack/repos/builtin/packages/lua/package.py
+++ b/var/spack/repos/builtin/packages/lua/package.py
@@ -10,7 +10,6 @@ from llnl.util.symlink import readlink
import spack.build_environment
from spack.package import *
-from spack.util.executable import Executable
# This is the template for a pkgconfig file for rpm
# https://github.com/guix-mirror/guix/raw/dcaf70897a0bad38a4638a2905aaa3c46b1f1402/gnu/packages/patches/lua-pkgconfig.patch
diff --git a/var/spack/repos/builtin/packages/magma/package.py b/var/spack/repos/builtin/packages/magma/package.py
index ee0397b66f..e97d663fd2 100644
--- a/var/spack/repos/builtin/packages/magma/package.py
+++ b/var/spack/repos/builtin/packages/magma/package.py
@@ -4,6 +4,7 @@
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
+import spack.util.environment
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/mapl/package.py b/var/spack/repos/builtin/packages/mapl/package.py
index ae8ce37805..bb9941f43d 100644
--- a/var/spack/repos/builtin/packages/mapl/package.py
+++ b/var/spack/repos/builtin/packages/mapl/package.py
@@ -5,6 +5,7 @@
import subprocess
+import spack.compiler
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/mesa/package.py b/var/spack/repos/builtin/packages/mesa/package.py
index 61494f3db6..8bf240409d 100644
--- a/var/spack/repos/builtin/packages/mesa/package.py
+++ b/var/spack/repos/builtin/packages/mesa/package.py
@@ -5,6 +5,7 @@
import sys
import spack.build_systems.meson
+import spack.variant
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/mmg/package.py b/var/spack/repos/builtin/packages/mmg/package.py
index 38a8123516..9bd3d63cef 100644
--- a/var/spack/repos/builtin/packages/mmg/package.py
+++ b/var/spack/repos/builtin/packages/mmg/package.py
@@ -7,7 +7,6 @@ import os
import spack.build_systems.cmake
from spack.package import *
-from spack.util.executable import which
class Mmg(CMakePackage):
diff --git a/var/spack/repos/builtin/packages/mpas-model/package.py b/var/spack/repos/builtin/packages/mpas-model/package.py
index 927026ddf9..39a6dfa741 100644
--- a/var/spack/repos/builtin/packages/mpas-model/package.py
+++ b/var/spack/repos/builtin/packages/mpas-model/package.py
@@ -5,7 +5,6 @@
import os
from spack.package import *
-from spack.util.executable import Executable
class MpasModel(MakefilePackage):
diff --git a/var/spack/repos/builtin/packages/mpich/package.py b/var/spack/repos/builtin/packages/mpich/package.py
index 5fc0538f15..b93fae3ddd 100644
--- a/var/spack/repos/builtin/packages/mpich/package.py
+++ b/var/spack/repos/builtin/packages/mpich/package.py
@@ -7,6 +7,7 @@ import os
import re
import sys
+import spack.compilers
from spack.build_environment import dso_suffix
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/msvc/package.py b/var/spack/repos/builtin/packages/msvc/package.py
index d58dea2f6b..4d25b87118 100644
--- a/var/spack/repos/builtin/packages/msvc/package.py
+++ b/var/spack/repos/builtin/packages/msvc/package.py
@@ -4,6 +4,7 @@
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
import re
+import spack.compiler
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/mvapich2/package.py b/var/spack/repos/builtin/packages/mvapich2/package.py
index 23129c3ec0..c0ff3bb07f 100644
--- a/var/spack/repos/builtin/packages/mvapich2/package.py
+++ b/var/spack/repos/builtin/packages/mvapich2/package.py
@@ -7,6 +7,7 @@ import os.path
import re
import sys
+import spack.compilers
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/nasm/package.py b/var/spack/repos/builtin/packages/nasm/package.py
index c3d64e4757..4afa6ebe1c 100644
--- a/var/spack/repos/builtin/packages/nasm/package.py
+++ b/var/spack/repos/builtin/packages/nasm/package.py
@@ -5,6 +5,7 @@
import glob
import os
+import spack.build_systems.generic
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/netcdf-cxx4/package.py b/var/spack/repos/builtin/packages/netcdf-cxx4/package.py
index 046c17a3fc..e5e3d0dc05 100644
--- a/var/spack/repos/builtin/packages/netcdf-cxx4/package.py
+++ b/var/spack/repos/builtin/packages/netcdf-cxx4/package.py
@@ -57,7 +57,7 @@ class NetcdfCxx4(CMakePackage):
return libs
msg = "Unable to recursively locate {0} {1} libraries in {2}"
- raise spack.error.NoLibrariesError(
+ raise NoLibrariesError(
msg.format("shared" if shared else "static", self.spec.name, self.spec.prefix)
)
diff --git a/var/spack/repos/builtin/packages/netcdf-fortran/package.py b/var/spack/repos/builtin/packages/netcdf-fortran/package.py
index ec74f25fc4..ba33ee5128 100644
--- a/var/spack/repos/builtin/packages/netcdf-fortran/package.py
+++ b/var/spack/repos/builtin/packages/netcdf-fortran/package.py
@@ -112,7 +112,7 @@ class NetcdfFortran(AutotoolsPackage):
return libs
msg = "Unable to recursively locate {0} {1} libraries in {2}"
- raise spack.error.NoLibrariesError(
+ raise NoLibrariesError(
msg.format("shared" if shared else "static", self.spec.name, self.spec.prefix)
)
diff --git a/var/spack/repos/builtin/packages/ninja-fortran/package.py b/var/spack/repos/builtin/packages/ninja-fortran/package.py
index 071a64b7b6..02fdac7128 100644
--- a/var/spack/repos/builtin/packages/ninja-fortran/package.py
+++ b/var/spack/repos/builtin/packages/ninja-fortran/package.py
@@ -3,8 +3,8 @@
#
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
+import spack.version
from spack.package import *
-from spack.util.executable import which_string
class NinjaFortran(Package):
diff --git a/var/spack/repos/builtin/packages/ninja/package.py b/var/spack/repos/builtin/packages/ninja/package.py
index 64e9eb3603..2249fcdbdb 100644
--- a/var/spack/repos/builtin/packages/ninja/package.py
+++ b/var/spack/repos/builtin/packages/ninja/package.py
@@ -5,7 +5,6 @@
import sys
from spack.package import *
-from spack.util.executable import which_string
class Ninja(Package):
diff --git a/var/spack/repos/builtin/packages/openmpi/package.py b/var/spack/repos/builtin/packages/openmpi/package.py
index 75d09bbc64..889d903435 100644
--- a/var/spack/repos/builtin/packages/openmpi/package.py
+++ b/var/spack/repos/builtin/packages/openmpi/package.py
@@ -11,6 +11,8 @@ import sys
import llnl.util.tty as tty
+import spack.compilers
+import spack.version
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/papi/package.py b/var/spack/repos/builtin/packages/papi/package.py
index 4ce382af5f..8d068108b3 100644
--- a/var/spack/repos/builtin/packages/papi/package.py
+++ b/var/spack/repos/builtin/packages/papi/package.py
@@ -9,6 +9,7 @@ import sys
import llnl.util.filesystem as fs
+import spack.util.environment
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/parallel-netcdf/package.py b/var/spack/repos/builtin/packages/parallel-netcdf/package.py
index f7bf693577..67b7cd867b 100644
--- a/var/spack/repos/builtin/packages/parallel-netcdf/package.py
+++ b/var/spack/repos/builtin/packages/parallel-netcdf/package.py
@@ -105,7 +105,7 @@ class ParallelNetcdf(AutotoolsPackage):
msg = f"Unable to recursively locate {'shared' if shared else 'static'} \
{self.spec.name} libraries in {self.spec.prefix}"
- raise spack.error.NoLibrariesError(msg)
+ raise NoLibrariesError(msg)
@when("@master")
def autoreconf(self, spec, prefix):
diff --git a/var/spack/repos/builtin/packages/pcre2/package.py b/var/spack/repos/builtin/packages/pcre2/package.py
index df79accc6e..67f0566385 100644
--- a/var/spack/repos/builtin/packages/pcre2/package.py
+++ b/var/spack/repos/builtin/packages/pcre2/package.py
@@ -3,6 +3,8 @@
#
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
+import spack.build_systems.autotools
+import spack.build_systems.cmake
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/perl/package.py b/var/spack/repos/builtin/packages/perl/package.py
index 2246a6ea9a..8b608e04d2 100644
--- a/var/spack/repos/builtin/packages/perl/package.py
+++ b/var/spack/repos/builtin/packages/perl/package.py
@@ -264,7 +264,7 @@ class Perl(Package): # Perl doesn't use Autotools, it should subclass Package
@classmethod
def determine_version(cls, exe):
- perl = spack.util.executable.Executable(exe)
+ perl = Executable(exe)
output = perl("--version", output=str, error=str)
if output:
match = re.search(r"perl.*\(v([0-9.]+)\)", output)
@@ -275,7 +275,7 @@ class Perl(Package): # Perl doesn't use Autotools, it should subclass Package
@classmethod
def determine_variants(cls, exes, version):
for exe in exes:
- perl = spack.util.executable.Executable(exe)
+ perl = Executable(exe)
output = perl("-V", output=str, error=str)
variants = ""
if output:
diff --git a/var/spack/repos/builtin/packages/phist/package.py b/var/spack/repos/builtin/packages/phist/package.py
index da926898e8..9b5c05b981 100644
--- a/var/spack/repos/builtin/packages/phist/package.py
+++ b/var/spack/repos/builtin/packages/phist/package.py
@@ -323,7 +323,7 @@ class Phist(CMakePackage):
tty.warn("========================== %s =======================" % hint)
try:
make("check")
- except spack.util.executable.ProcessError:
+ except ProcessError:
raise InstallError("run-test of phist ^mpich: Hint: " + hint)
else:
make("check")
diff --git a/var/spack/repos/builtin/packages/pixman/package.py b/var/spack/repos/builtin/packages/pixman/package.py
index e84ff0a48a..dc982be2ad 100644
--- a/var/spack/repos/builtin/packages/pixman/package.py
+++ b/var/spack/repos/builtin/packages/pixman/package.py
@@ -5,6 +5,8 @@
import sys
+import spack.build_systems.autotools
+import spack.build_systems.meson
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/py-matplotlib/package.py b/var/spack/repos/builtin/packages/py-matplotlib/package.py
index 04b5d63447..28db0d5fa5 100644
--- a/var/spack/repos/builtin/packages/py-matplotlib/package.py
+++ b/var/spack/repos/builtin/packages/py-matplotlib/package.py
@@ -332,7 +332,7 @@ class PyMatplotlib(PythonPackage):
include.extend(query.headers.directories)
try:
library.extend(query.libs.directories)
- except spack.error.NoLibrariesError:
+ except NoLibrariesError:
pass
# Build uses a mix of Spack's compiler wrapper and the actual compiler,
diff --git a/var/spack/repos/builtin/packages/py-pennylane-lightning-kokkos/package.py b/var/spack/repos/builtin/packages/py-pennylane-lightning-kokkos/package.py
index 4513912e67..34b01dac5c 100644
--- a/var/spack/repos/builtin/packages/py-pennylane-lightning-kokkos/package.py
+++ b/var/spack/repos/builtin/packages/py-pennylane-lightning-kokkos/package.py
@@ -2,6 +2,7 @@
# Spack Project Developers. See the top-level COPYRIGHT file for details.
#
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
+import spack.build_systems.cmake
from spack.build_systems.python import PythonPipBuilder
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/py-pennylane-lightning/package.py b/var/spack/repos/builtin/packages/py-pennylane-lightning/package.py
index 383e03a2d2..3e950aad45 100644
--- a/var/spack/repos/builtin/packages/py-pennylane-lightning/package.py
+++ b/var/spack/repos/builtin/packages/py-pennylane-lightning/package.py
@@ -4,6 +4,7 @@
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
+import spack.build_systems.cmake
from spack.build_systems.python import PythonPipBuilder
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/python/package.py b/var/spack/repos/builtin/packages/python/package.py
index 3bfdc240d0..eedd0067f5 100644
--- a/var/spack/repos/builtin/packages/python/package.py
+++ b/var/spack/repos/builtin/packages/python/package.py
@@ -16,6 +16,7 @@ from typing import Dict, List
import llnl.util.tty as tty
from llnl.util.lang import dedupe
+import spack.paths
from spack.build_environment import dso_suffix, stat_suffix
from spack.package import *
from spack.util.prefix import Prefix
@@ -1088,7 +1089,7 @@ print(json.dumps(config))
if lib:
return lib
- raise spack.error.NoLibrariesError(
+ raise NoLibrariesError(
"Unable to find {} libraries with the following names:\n\n* ".format(self.name)
+ "\n* ".join(candidates)
)
@@ -1114,7 +1115,7 @@ print(json.dumps(config))
config_h = headers[0]
break
else:
- raise spack.error.NoHeadersError(
+ raise NoHeadersError(
"Unable to locate {} headers in any of these locations:\n\n* ".format(self.name)
+ "\n* ".join(candidates)
)
diff --git a/var/spack/repos/builtin/packages/root/package.py b/var/spack/repos/builtin/packages/root/package.py
index e6da973906..d8bb48c720 100644
--- a/var/spack/repos/builtin/packages/root/package.py
+++ b/var/spack/repos/builtin/packages/root/package.py
@@ -7,6 +7,7 @@
import os
import sys
+import spack.util.environment
from spack.operating_systems.mac_os import macos_version
from spack.package import *
from spack.util.environment import is_system_path
diff --git a/var/spack/repos/builtin/packages/rpcsvc-proto/package.py b/var/spack/repos/builtin/packages/rpcsvc-proto/package.py
index cd18cc8dd5..2aa4338cf1 100644
--- a/var/spack/repos/builtin/packages/rpcsvc-proto/package.py
+++ b/var/spack/repos/builtin/packages/rpcsvc-proto/package.py
@@ -3,6 +3,7 @@
#
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
+import spack.paths
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/sccache/package.py b/var/spack/repos/builtin/packages/sccache/package.py
index 23fbfe8b77..2f526242ba 100644
--- a/var/spack/repos/builtin/packages/sccache/package.py
+++ b/var/spack/repos/builtin/packages/sccache/package.py
@@ -7,6 +7,7 @@ import os
import re
import spack.build_systems
+import spack.build_systems.cargo
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/seqkit/package.py b/var/spack/repos/builtin/packages/seqkit/package.py
index 639f7cbbaf..da1e1dbae8 100644
--- a/var/spack/repos/builtin/packages/seqkit/package.py
+++ b/var/spack/repos/builtin/packages/seqkit/package.py
@@ -3,6 +3,7 @@
#
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
+import spack.build_systems.go
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/serialbox/package.py b/var/spack/repos/builtin/packages/serialbox/package.py
index 1d756cb0d9..ed455611cf 100644
--- a/var/spack/repos/builtin/packages/serialbox/package.py
+++ b/var/spack/repos/builtin/packages/serialbox/package.py
@@ -131,7 +131,7 @@ class Serialbox(CMakePackage):
return libs
msg = "Unable to recursively locate {0} libraries in {1}"
- raise spack.error.NoLibrariesError(msg.format(self.spec.name, self.spec.prefix))
+ raise NoLibrariesError(msg.format(self.spec.name, self.spec.prefix))
def flag_handler(self, name, flags):
cmake_flags = []
diff --git a/var/spack/repos/builtin/packages/sherpa/package.py b/var/spack/repos/builtin/packages/sherpa/package.py
index 7d0010a7c2..20133e54be 100644
--- a/var/spack/repos/builtin/packages/sherpa/package.py
+++ b/var/spack/repos/builtin/packages/sherpa/package.py
@@ -3,6 +3,8 @@
#
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
+import spack.build_systems.autotools
+import spack.build_systems.cmake
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/singularity-eos/package.py b/var/spack/repos/builtin/packages/singularity-eos/package.py
index ef5c571118..46a768ce47 100644
--- a/var/spack/repos/builtin/packages/singularity-eos/package.py
+++ b/var/spack/repos/builtin/packages/singularity-eos/package.py
@@ -5,6 +5,7 @@
import os
+import spack.version
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/singularityce/package.py b/var/spack/repos/builtin/packages/singularityce/package.py
index 08630bf751..6a14fe471f 100644
--- a/var/spack/repos/builtin/packages/singularityce/package.py
+++ b/var/spack/repos/builtin/packages/singularityce/package.py
@@ -8,6 +8,7 @@ import shutil
import llnl.util.tty as tty
+import spack.tengine
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/spectrum-mpi/package.py b/var/spack/repos/builtin/packages/spectrum-mpi/package.py
index 54c70c6b56..e36be9826c 100644
--- a/var/spack/repos/builtin/packages/spectrum-mpi/package.py
+++ b/var/spack/repos/builtin/packages/spectrum-mpi/package.py
@@ -5,6 +5,7 @@
import os
import re
+import spack.compilers
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/sqlite/package.py b/var/spack/repos/builtin/packages/sqlite/package.py
index 57f1272801..ef8840ea7a 100644
--- a/var/spack/repos/builtin/packages/sqlite/package.py
+++ b/var/spack/repos/builtin/packages/sqlite/package.py
@@ -7,6 +7,8 @@ import re
import sys
from tempfile import NamedTemporaryFile
+import spack.build_systems.autotools
+import spack.build_systems.nmake
import spack.platforms
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/strumpack/package.py b/var/spack/repos/builtin/packages/strumpack/package.py
index 28cc57f26d..d1cb572f29 100644
--- a/var/spack/repos/builtin/packages/strumpack/package.py
+++ b/var/spack/repos/builtin/packages/strumpack/package.py
@@ -9,7 +9,6 @@ import llnl.util.tty as tty
from spack.package import *
from spack.util.environment import set_env
-from spack.util.executable import ProcessError
class Strumpack(CMakePackage, CudaPackage, ROCmPackage):
diff --git a/var/spack/repos/builtin/packages/szx/package.py b/var/spack/repos/builtin/packages/szx/package.py
index 9d05663fce..df430e0528 100644
--- a/var/spack/repos/builtin/packages/szx/package.py
+++ b/var/spack/repos/builtin/packages/szx/package.py
@@ -2,6 +2,8 @@
# Spack Project Developers. See the top-level COPYRIGHT file for details.
#
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
+import spack.build_systems.autotools
+import spack.build_systems.cmake
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/tcl/package.py b/var/spack/repos/builtin/packages/tcl/package.py
index a6b243d4bf..b8e9be6e89 100644
--- a/var/spack/repos/builtin/packages/tcl/package.py
+++ b/var/spack/repos/builtin/packages/tcl/package.py
@@ -8,6 +8,8 @@ import sys
from llnl.util.filesystem import find_first
+import spack.build_systems.autotools
+import spack.build_systems.nmake
from spack.package import *
from spack.util.environment import is_system_path
diff --git a/var/spack/repos/builtin/packages/thrust/package.py b/var/spack/repos/builtin/packages/thrust/package.py
index 5d70406dc6..bbd1708b8c 100644
--- a/var/spack/repos/builtin/packages/thrust/package.py
+++ b/var/spack/repos/builtin/packages/thrust/package.py
@@ -3,6 +3,8 @@
#
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
+import spack.build_systems.cmake
+import spack.build_systems.generic
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/trilinos/package.py b/var/spack/repos/builtin/packages/trilinos/package.py
index 0669adff63..bb12c6caa3 100644
--- a/var/spack/repos/builtin/packages/trilinos/package.py
+++ b/var/spack/repos/builtin/packages/trilinos/package.py
@@ -9,7 +9,6 @@ import re
import sys
from spack.build_environment import dso_suffix
-from spack.error import NoHeadersError
from spack.operating_systems.mac_os import macos_version
from spack.package import *
from spack.pkg.builtin.kokkos import Kokkos
diff --git a/var/spack/repos/builtin/packages/upcxx/package.py b/var/spack/repos/builtin/packages/upcxx/package.py
index 9197102568..141060a9d5 100644
--- a/var/spack/repos/builtin/packages/upcxx/package.py
+++ b/var/spack/repos/builtin/packages/upcxx/package.py
@@ -6,6 +6,7 @@
import os
import re
+import spack.platforms
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/util-macros/package.py b/var/spack/repos/builtin/packages/util-macros/package.py
index 1f6411e2bd..e0bf69fb56 100644
--- a/var/spack/repos/builtin/packages/util-macros/package.py
+++ b/var/spack/repos/builtin/packages/util-macros/package.py
@@ -3,6 +3,7 @@
#
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
+import spack.url
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/wayland-protocols/package.py b/var/spack/repos/builtin/packages/wayland-protocols/package.py
index 20b113de24..dc5054a356 100644
--- a/var/spack/repos/builtin/packages/wayland-protocols/package.py
+++ b/var/spack/repos/builtin/packages/wayland-protocols/package.py
@@ -3,6 +3,7 @@
#
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
+import spack.build_systems.meson
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/yafyaml/package.py b/var/spack/repos/builtin/packages/yafyaml/package.py
index 996b57fa1e..379fff05ab 100644
--- a/var/spack/repos/builtin/packages/yafyaml/package.py
+++ b/var/spack/repos/builtin/packages/yafyaml/package.py
@@ -6,6 +6,7 @@
import os
import re
+import spack.compiler
from spack.package import *
diff --git a/var/spack/repos/builtin/packages/zziplib/package.py b/var/spack/repos/builtin/packages/zziplib/package.py
index d91c0cf2ce..2a3b10a14d 100644
--- a/var/spack/repos/builtin/packages/zziplib/package.py
+++ b/var/spack/repos/builtin/packages/zziplib/package.py
@@ -4,7 +4,6 @@
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
import spack.build_systems.autotools
-import spack.build_systems.cmake
from spack.package import *