diff options
| author | Lasse Collin <lasse.collin@tukaani.org> | 2011-03-24 01:21:32 +0200 |
|---|---|---|
| committer | Lasse Collin <lasse.collin@tukaani.org> | 2011-03-24 01:24:12 +0200 |
| commit | b441d39855516ae618faffd5156261b8b413394f (patch) | |
| tree | 46140610a615ec68a9d07cb0059f56a377fc388d | |
| parent | xz: Clean up suffix.c. (diff) | |
| download | xz-b441d39855516ae618faffd5156261b8b413394f.tar.xz | |
Scripts: Fix xzgrep -l.
It didn't work at all. It tried to use the -q option
for grep, but it appended it after "--". This works
around it by redirecting to /dev/null. The downside
is that this can be slower with big files compared
to proper use of "grep -q".
Thanks to Gregory Margo.
| -rw-r--r-- | src/scripts/xzgrep.in | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/scripts/xzgrep.in b/src/scripts/xzgrep.in index bfa9fb53..18f780a9 100644 --- a/src/scripts/xzgrep.in +++ b/src/scripts/xzgrep.in @@ -156,9 +156,9 @@ for i; do exec 5>&1 ($uncompress -- "$i" 5>&-; echo $? >&5) 3>&- | if test $files_with_matches -eq 1; then - eval "$grep" -q && { printf '%s\n' "$i" || exit 2; } + eval "$grep" >/dev/null && { printf '%s\n' "$i" || exit 2; } elif test $files_without_matches -eq 1; then - eval "$grep" -q || { + eval "$grep" >/dev/null || { r=$? if test $r -eq 1; then printf '%s\n' "$i" || r=2 |