Skip to content

Commit 1a78f75

Browse files
OriPekelmanclaude
andcommitted
reprobe: guard probe exit codes inside the xargs lambda
A probe exiting 255 makes xargs abort the entire fan-out — one gem killed the 57af7f9 sweep at 13k/189k probes. || true inside the lambda: one bad gem costs one verdict, never the sweep. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent ea6f55a commit 1a78f75

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

bin/reprobe-corpus.sh

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,10 +102,13 @@ SP_REV="$FULL_REV" SP_LEDGER="$SPINEL_COMPAT_LEDGER" ruby -e '
102102
'
103103

104104
# Fan out: one `probe --dir` per cached gem, all appending to the ledger.
105+
# NB: no `exec` + a trailing `|| true` INSIDE the lambda — a probe exiting 255
106+
# makes xargs abort the ENTIRE fan-out (killed the 57af7f9 run at 13k/189k).
107+
# One bad gem must cost one verdict, never the sweep.
105108
cat "$TSV" | xargs -P "$SHARDS" -d'\n' -I{} bash -c '
106109
IFS=$'"'"'\t'"'"' read -r g v d <<< "$1"
107110
ulimit -v 6291456
108-
exec "$0" probe "$g" "$v" --dir "$d" >/dev/null 2>&1
111+
"$0" probe "$g" "$v" --dir "$d" >/dev/null 2>&1 || true
109112
' "$CLI" {} || true
110113

111114
# Aggregate report (pure ledger aggregation — no probes, no network).

0 commit comments

Comments
 (0)