Thomas Bushnell, BSG wrote:
> Create a gzipped file:
>
> $ for i in $(seq 1 100); do echo $i; done > file.txt
> $ gzip file.txt
>
> Then:
>
> $ zgrep -9 17 file.txt.gz
> 8
> 9
> ...
> 25
> 26
>
> works. But:
>
> $ zgrep -10 17 file.txt.gz
> gzip: 17.gz: No such file or directory
>
> fails.
>
> Note that "grep -9 17 file.txt" and "grep -10 17 file.txt" work just
> fine.
>
> (https://bugs.launchpad.net/bugs/1032831)
>
> Please include me in the CC on any replies.
Hi Thomas,
Thanks for the report.
That made a nice puzzle to go with my morning caffeine ;-)
Here's the fix:
I'll write the NEWS entry and test later.
diff --git a/zgrep.in b/zgrep.in
index d09bfa7..abc5847 100644
--- a/zgrep.in
+++ b/zgrep.in
@@ -66,9 +66,13 @@ while test $# -ne 0; do
case $option in
(-[0123456789EFGHIKLPRTUVZabchilnoqrsuvwxyz]?*)
- arg2=-\'$(expr "X$option" : 'X-.[0-9]*\(.*\)' | sed "$escape")
- eval "set -- $arg2 "'${1+"$@"}'
- option=$(expr "X$option" : 'X\(-.[0-9]*\)');;
+ if expr "X$option" : 'X-[0-9]\+$' > /dev/null; then
+ : # Let a multi-digit, digit-only option like -10 fall through.
+ else
+ arg2=-\'$(expr "X$option" : 'X-.[0-9]*\(.*\)' | sed "$escape")
+ eval "set -- $arg2 "'${1+"$@"}'
+ option=$(expr "X$option" : 'X\(-.[0-9]*\)')
+ fi;;
(--binary-*=* | --[lm]a*=* | --reg*=*)
;;
(-[ABCDXdefm] | binary-* | --file | --[lm]a* | --reg*)
