diff options
author | Lasse Collin <lasse.collin@tukaani.org> | 2021-01-11 23:28:52 +0200 |
---|---|---|
committer | Lasse Collin <lasse.collin@tukaani.org> | 2022-07-12 18:30:56 +0300 |
commit | d655b8c9cb5a4e596c0fa2290fa91e995b734f97 (patch) | |
tree | 457a3151dbb3f96dcae3370552ea5115b7ff08a8 /tests/test_bcj_exact_size.c | |
parent | Scripts: Fix exit status of xzdiff/xzcmp. (diff) | |
download | xz-d655b8c9cb5a4e596c0fa2290fa91e995b734f97.tar.xz |
Scripts: Fix exit status of xzgrep.
Omit the -q option from xz, gzip, and bzip2. With xz this shouldn't
matter. With gzip it's important because -q makes gzip replace SIGPIPE
with exit status 2. With bzip2 it's important because with -q bzip2
is completely silent if input is corrupt while other decompressors
still give an error message.
Avoiding exit status 2 from gzip is important because bzip2 uses
exit status 2 to indicate corrupt input. Before this commit xzgrep
didn't recognize corrupt .bz2 files because xzgrep was treating
exit status 2 as SIGPIPE for gzip compatibility.
zstd still needs -q because otherwise it is noisy in normal
operation.
The code to detect real SIGPIPE didn't check if the exit status
was due to a signal (>= 128) and so could ignore some other exit
status too.
Diffstat (limited to 'tests/test_bcj_exact_size.c')
0 files changed, 0 insertions, 0 deletions