From 686405ef416955621a89815e07cb64e1ee4f1495 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Th=C3=A9ophane=20Hufschmitt?= <7226587+thufschmitt@users.noreply.github.com> Date: Wed, 6 Mar 2024 22:36:37 +0100 Subject: [PATCH] Fix sudo in the darwin installer (#10128) --- scripts/install-multi-user.sh | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/scripts/install-multi-user.sh b/scripts/install-multi-user.sh index 1dbb93bf9..4d6a1914e 100644 --- a/scripts/install-multi-user.sh +++ b/scripts/install-multi-user.sh @@ -69,16 +69,17 @@ readonly PROXY_ENVIRONMENT_VARIABLES=( NO_PROXY ) -SUDO_EXTRA_ENVIRONMENT_VARIABLES=() +SUDO_KEPT_ENVIRONMENT_VARIABLES="" setup_sudo_extra_environment_variables() { - local i=${#SUDO_EXTRA_ENVIRONMENT_VARIABLES[@]} for variable in "${PROXY_ENVIRONMENT_VARIABLES[@]}"; do if [ "x${!variable:-}" != "x" ]; then - SUDO_EXTRA_ENVIRONMENT_VARIABLES[i]="$variable=${!variable}" - i=$((i + 1)) + SUDO_KEPT_ENVIRONMENT_VARIABLES="$SUDO_KEPT_ENVIRONMENT_VARIABLES,$variable" fi done + + # Required by the darwin installer + export SUDO_KEPT_ENVIRONMENT_VARIABLES } setup_sudo_extra_environment_variables @@ -386,7 +387,7 @@ _sudo() { if is_root; then env "$@" else - sudo "${SUDO_EXTRA_ENVIRONMENT_VARIABLES[@]}" "$@" + sudo --preserve-env="$SUDO_KEPT_ENVIRONMENT_VARIABLES" "$@" fi }