summaryrefslogtreecommitdiff
path: root/lib
diff options
context:
space:
mode:
authorTodd Gamblin <tgamblin@llnl.gov>2020-04-20 08:32:03 -0700
committerGitHub <noreply@github.com>2020-04-20 08:32:03 -0700
commitde13ea228407c377b6c42b54f70a17bef8be1854 (patch)
tree6d7de576857961ef01273e702da794f250b8ad84 /lib
parente9e2d1d3cab1352e0b2dbaafb31aaaff499e6079 (diff)
downloadspack-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.rst13
-rw-r--r--lib/spack/spack/cmd/ci.py2
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')