From db1c84d3b019fc5f0d6486a4149320e1c73e4646 Mon Sep 17 00:00:00 2001 From: Meta Vens Date: Wed, 23 Sep 2026 08:13:33 +0200 Subject: [PATCH] Record two rakes in the stopgap's header A job piped into tail loses its exit status, because the command runs under bash -c and the sentinel then reports the pipeline's zero rather than the job's. That is the same false-green the CRLF fix closed, arriving by a different road, and it cost a Soul run today. The second is smaller and just as costly to rediscover: CultLib has no Cargo.toml at its root, so a rust job must cd into the package first or cargo fails in a way that looks like a broken image. Co-Authored-By: Claude Opus 5 --- tools/stopgap/ygg-verify.sh | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/tools/stopgap/ygg-verify.sh b/tools/stopgap/ygg-verify.sh index 0d4982e..3982127 100644 --- a/tools/stopgap/ygg-verify.sh +++ b/tools/stopgap/ygg-verify.sh @@ -32,6 +32,14 @@ # There is no hand-written mutation harness: the operator retired it on # 2026-09-22. Mutation testing uses the ecosystem's tools on a cut's diff. # +# TWO RAKES, both paid for on 2026-09-23: +# * The command runs under `bash -c`, so piping the WHOLE command into tail or +# grep discards its exit status and the verdict sentinel faithfully reports +# the pipeline's zero. Keep the job's own exit status last: run the command, +# then filter its saved output, or use `set -o pipefail` inside the command. +# * CultLib has no Cargo.toml at its root. A rust job must `cd packages/` +# first, or cargo fails in a way that looks like the image is wrong. +# # EDITING: bash reads a running script from disk as it goes, so editing this # file in place breaks every job still running it (one died with "unexpected # EOF", 2026-09-22). Write the new version to a temp file and rename it over