diff options
author | Todd Gamblin <tgamblin@llnl.gov> | 2020-04-20 08:32:03 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-04-20 08:32:03 -0700 |
commit | de13ea228407c377b6c42b54f70a17bef8be1854 (patch) | |
tree | 6d7de576857961ef01273e702da794f250b8ad84 /lib | |
parent | e9e2d1d3cab1352e0b2dbaafb31aaaff499e6079 (diff) | |
download | spack-de13ea228407c377b6c42b54f70a17bef8be1854.tar.gz spack-de13ea228407c377b6c42b54f70a17bef8be1854.tar.bz2 spack-de13ea228407c377b6c42b54f70a17bef8be1854.tar.xz spack-de13ea228407c377b6c42b54f70a17bef8be1854.zip |
autotools: remove FORCE_UNSAFE_CONFIGURE from docs, commands, code (#16145)
Since #16132, we've consolidated the setting of FORCE_UNSAFE_CONFIGURE to
`autotools.py`, so we don't need to use it in packages like `coreutils`,
in our commands, or in our container recipes.
- [x] Remove FORCE_UNSAFE_CONFIGURE from packages
- [x] Remove FORCE_UNSAFE_CONFIGURE from container recipes
- [x] Remove FORCE_UNSAFE_CONFIGURE from `spack ci` command
Diffstat (limited to 'lib')
-rw-r--r-- | lib/spack/docs/workflows.rst | 13 | ||||
-rw-r--r-- | lib/spack/spack/cmd/ci.py | 2 |
2 files changed, 4 insertions, 11 deletions
diff --git a/lib/spack/docs/workflows.rst b/lib/spack/docs/workflows.rst index 9ce664b6ca..239ee55a6a 100644 --- a/lib/spack/docs/workflows.rst +++ b/lib/spack/docs/workflows.rst @@ -284,9 +284,9 @@ have some drawbacks: The ``spack load`` and ``spack module tcl loads`` commands, on the other hand, are not very smart: if the user-supplied spec matches more than one installed package, then ``spack module tcl loads`` will - fail. This default behavior may change in the future. For now, - the workaround is to either be more specific on any failing ``spack load`` - commands or to use ``spack load --first`` to allow spack to load the + fail. This default behavior may change in the future. For now, + the workaround is to either be more specific on any failing ``spack load`` + commands or to use ``spack load --first`` to allow spack to load the first matching spec. @@ -1432,10 +1432,6 @@ The following functionality is prepared: #. Base image: the example starts from a minimal ubuntu. -#. Installing as root: docker images are usually set up as root. - Since some autotools scripts might complain about this being unsafe, we set - ``FORCE_UNSAFE_CONFIGURE=1`` to avoid configure errors. - #. Pre-install the spack dependencies, including modules from the packages. This avoids needing to build those from scratch via ``spack bootstrap``. Package installs are followed by a clean-up of the system package index, @@ -1466,8 +1462,7 @@ In order to build and run the image, execute: # general environment for docker ENV DEBIAN_FRONTEND=noninteractive \ - SPACK_ROOT=/usr/local \ - FORCE_UNSAFE_CONFIGURE=1 + SPACK_ROOT=/usr/local # install minimal spack dependencies RUN apt-get update \ diff --git a/lib/spack/spack/cmd/ci.py b/lib/spack/spack/cmd/ci.py index 22c555d85a..e87da8c8af 100644 --- a/lib/spack/spack/cmd/ci.py +++ b/lib/spack/spack/cmd/ci.py @@ -283,8 +283,6 @@ def ci_rebuild(args): spack_cmd = exe.which('spack') - os.environ['FORCE_UNSAFE_CONFIGURE'] = '1' - cdash_report_dir = os.path.join(ci_artifact_dir, 'cdash_report') temp_dir = os.path.join(ci_artifact_dir, 'jobs_scratch_dir') job_log_dir = os.path.join(temp_dir, 'logs') |