colin/nixpkgs
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

stdenv-setup: Remove any declare -g

Browse Source 
This is invalid before bash-4.2, affecting bash used impurely in
nix-shell on MacOS.
This commit is contained in:
John Ericson 2017-07-25 20:58:47 -04:00
parent ea7d13cf1a
commit 820e4021d3
1 changed files with 4 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
pkgs/stdenv/generic/setup.sh
View File

@ -285,10 +285,7 @@ findInputs() {
*" $pkg "*) return 0 ;;
esac
# For some reason, bash gives us some (hopefully limited) eval
# "for free"! Everything is single-quoted except for `"$var"`
# so `var` is expanded first.
declare -g "$var"'=("${'"$var"'[@]}" "$pkg")'
eval "$var"'+=("$pkg")'
if ! [ -e "$pkg" ]; then
echo "build input $pkg does not exist" >&2
@ -318,19 +315,19 @@ findInputs() {
if [ -z "$crossConfig" ]; then
# Not cross-compiling - both buildInputs (and variants like propagatedBuildInputs)
# are handled identically to nativeBuildInputs
declare -ga nativePkgs
declare -a nativePkgs
for i in $nativeBuildInputs $buildInputs \
$defaultNativeBuildInputs $defaultBuildInputs \
$propagatedNativeBuildInputs $propagatedBuildInputs; do
findInputs "$i" nativePkgs propagated-native-build-inputs
done
else
declare -ga crossPkgs
declare -a crossPkgs
for i in $buildInputs $defaultBuildInputs $propagatedBuildInputs; do
findInputs "$i" crossPkgs propagated-build-inputs
done
declare -ga nativePkgs
declare -a nativePkgs
for i in $nativeBuildInputs $defaultNativeBuildInputs $propagatedNativeBuildInputs; do
findInputs "$i" nativePkgs propagated-native-build-inputs
done