Skip to content

Commit eed3af5

Browse files
sbryngelsonclaude
andcommitted
Fix bench build: use PIDs instead of job specs for wait
wait %1 and %2 require job control (set -m), which is disabled in non-interactive shells. nick-fields/retry runs commands in a non-interactive shell, causing 'no such job' (exit 127) even when both builds succeed. Use $! PIDs instead. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 4f39d1b commit eed3af5

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

.github/workflows/bench.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -108,8 +108,10 @@ jobs:
108108
timeout_minutes: 480
109109
command: |
110110
(cd pr && ${{ matrix.build_script }}) &
111+
pid1=$!
111112
(cd master && ${{ matrix.build_script }}) &
112-
wait %1 && wait %2
113+
pid2=$!
114+
wait $pid1 && wait $pid2
113115
on_retry_command: |
114116
(cd pr && ./mfc.sh clean) &
115117
(cd master && ./mfc.sh clean) &

0 commit comments

Comments
 (0)