From 9f8e99d287577eb9814925bf23ebf621ff040e9a Mon Sep 17 00:00:00 2001 From: chahinebrini Date: Sat, 30 May 2026 10:05:10 +0200 Subject: [PATCH] fix(deploy): preserve rc under set -e (run_quiet swallowed errors silently) 'wait $pid' triggered 'set -e' before 'local rc=$?' could capture exit code, so xcodebuild failures vanished without any log dump. Use 'wait $pid && rc=0 || rc=$?' idiom to keep the script alive and let the error-dump branch run. --- apps/rebreak-native/deploy.sh | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/apps/rebreak-native/deploy.sh b/apps/rebreak-native/deploy.sh index 3bce077..4a64081 100755 --- a/apps/rebreak-native/deploy.sh +++ b/apps/rebreak-native/deploy.sh @@ -174,7 +174,7 @@ run_quiet() { return ${PIPESTATUS[0]} fi local start=$SECONDS - local expected pid elapsed subtitle + local expected pid elapsed subtitle rc expected=$(runtime_lookup "$label" || echo 0) expected=${expected:-0} RUN_QUIET_I=0 @@ -203,8 +203,7 @@ run_quiet() { render_progress "$elapsed" "$expected" "$label" "$subtitle" sleep 0.2 done - wait "$pid" - local rc=$? + wait "$pid" && rc=0 || rc=$? elapsed=$((SECONDS - start)) # Clear progress line printf '\r\033[K' >&2