diff options
| author | Claude <noreply@anthropic.com> | 2026-09-21 19:21:32 +0000 |
|---|---|---|
| committer | Claude <noreply@anthropic.com> | 2026-09-21 19:21:32 +0000 |
| commit | 97e9b5e6f18705666b9d622b9eab0b31836321ba (patch) | |
| tree | bdca9c22b584f1400f266da6c1992e7967ec2e2f /bin/gradle-report | |
| parent | 95f95acdb551168f2e5a52e000ff62167be266a8 (diff) | |
publish-apk: fix a SIGPIPE this repo's own README warns about
The zip check used `unzip -l | grep -q`. Under pipefail, grep exits on its
first match and SIGPIPEs unzip, so the pipeline fails on a perfectly valid
file. It rejected a real 112MB APK as 'not an APK'.
That exact trap is in the README's table, written after it broke crash
detection in android-launch-check. Knowing about a trap is not the same as not
falling into it, which is the argument for these scripts existing at all.
Captures the listing first and tests it with `case`, and the new test builds a
300-entry zip so grep matches long before the listing ends -- which is when
SIGPIPE actually bites. A single-entry fixture would have passed either way.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018AwnALZL6bD4VBtATDVv3B
Diffstat (limited to 'bin/gradle-report')
0 files changed, 0 insertions, 0 deletions
