Skip to content

Commit

Permalink
spack-start.bash: do not set unnecessary variables (#17)
Browse files Browse the repository at this point in the history
* Since this script is dot sourced into the user's environment, we
  shouldn't set any temporary variables.
  • Loading branch information
harshula authored Dec 5, 2023
1 parent fe997a5 commit eb9e433
Showing 1 changed file with 1 addition and 6 deletions.
7 changes: 1 addition & 6 deletions spack-start.bash
Original file line number Diff line number Diff line change
Expand Up @@ -8,11 +8,6 @@
# $PREFIX/spack-packages
# $PREFIX/release (defined in $PREFIX/spack-config/*/*/config.yaml)

set -x
PREFIX="$(dirname $BASH_SOURCE)/.."
SPACK="$PREFIX/spack"
set +x

# https://spack-tutorial.readthedocs.io/en/latest/tutorial_configuration.html#yaml-format
#
# When Spack checks its configuration, the configuration scopes are updated
Expand All @@ -34,4 +29,4 @@ export SPACK_DISABLE_LOCAL_CONFIG="true"
# TODO: Decide on the version of Python that we approve.
#export SPACK_PYTHON=

. $SPACK/share/spack/setup-env.sh
. $(dirname $BASH_SOURCE)/../spack/share/spack/setup-env.sh

0 comments on commit eb9e433

Please sign in to comment.