diff options
| author | Craig Jennings <c@cjennings.net> | 2026-02-23 06:06:55 -0600 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-02-23 06:06:55 -0600 |
| commit | 2ef27b6a5d190c4d38eb4d13a039e10c006d7c7b (patch) | |
| tree | 00cc8eccd698785d53fe4541b334f3aa71651d5b /scripts/build-release | |
| parent | 1268a283526ca4b1d11d391ede944f055b4abafd (diff) | |
| download | archangel-2ef27b6a5d190c4d38eb4d13a039e10c006d7c7b.tar.gz archangel-2ef27b6a5d190c4d38eb4d13a039e10c006d7c7b.zip | |
chore: add set -euo pipefail to scripts for safety
Enable undefined variable checking (set -u) and pipefail across
standalone scripts. Guard SUDO_USER references with ${SUDO_USER:-}
for set -u compatibility.
Diffstat (limited to 'scripts/build-release')
| -rwxr-xr-x | scripts/build-release | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/scripts/build-release b/scripts/build-release index 2fbf004..9c6614e 100755 --- a/scripts/build-release +++ b/scripts/build-release @@ -12,13 +12,13 @@ # - $ARCHSETUP_INBOX (notification for test VM rebuild, if set) # - $DIST_REMOTE_HOST:$DIST_REMOTE_PATH (if set and reachable) -set -e +set -euo pipefail SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)" PROJECT_DIR="$(dirname "$SCRIPT_DIR")" # Get actual user (not root when running with sudo) -if [[ -n "$SUDO_USER" ]]; then +if [[ -n "${SUDO_USER:-}" ]]; then REAL_USER="$SUDO_USER" REAL_HOME=$(getent passwd "$SUDO_USER" | cut -d: -f6) else @@ -140,7 +140,7 @@ distribute_remote() { # Run SCP as the real user (not root) to use their SSH keys local scp_cmd="scp" - if [[ -n "$SUDO_USER" ]]; then + if [[ -n "${SUDO_USER:-}" ]]; then scp_cmd="sudo -u $SUDO_USER scp" fi |