From 093e60d1115893e84ca42b9da882f57bcdec56b3 Mon Sep 17 00:00:00 2001 From: Ricardo Pardini Date: Tue, 3 Jan 2023 13:07:40 +0100 Subject: [PATCH] armbian-next: `exec` into `sudo` when relaunching (AR-1452) - with `exec`, `sudo` replaces the current shell, and thus never returns --- lib/functions/cli/utils-cli.sh | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/lib/functions/cli/utils-cli.sh b/lib/functions/cli/utils-cli.sh index c09572bc94..5f47eb4876 100644 --- a/lib/functions/cli/utils-cli.sh +++ b/lib/functions/cli/utils-cli.sh @@ -211,8 +211,9 @@ function cli_standard_relaunch_docker_or_sudo() { display_alert "This script requires root privileges; Docker is unavailable" "trying to use sudo" "wrn" declare -g ARMBIAN_CLI_RELAUNCH_ARGS=() - produce_relaunch_parameters # produces ARMBIAN_CLI_RELAUNCH_ARGS - sudo --preserve-env "${SRC}/compile.sh" "${ARMBIAN_CLI_RELAUNCH_ARGS[@]}" # MARK: relaunch done here! + produce_relaunch_parameters # produces ARMBIAN_CLI_RELAUNCH_ARGS + # shellcheck disable=SC2093 # re-launching under sudo: replace the current shell, and never return. + exec sudo --preserve-env "${SRC}/compile.sh" "${ARMBIAN_CLI_RELAUNCH_ARGS[@]}" # MARK: relaunch done here! display_alert "AFTER SUDO!!!" "AFTER SUDO!!!" "warn" fi }