Package: gzip
Version: 1.3.5-13
Followup-For: Bug #292896

I note that it's a long time since this bug was reported, and my
one-line patch has still not been applied. I imagine that the problem
is that my fix uses a bash-specific feature, $PIPESTATUS.

I've now updated my patch with a (GPLed, hence same license) hack
found in the c.u.s. FAQ at http://home.comcast.net/~j.p.h/cus-faq-2

which gives the following diff with the current version of zgrep (from
gzip 1.3.5-13):

-----------cut here------------------
--- /bin/zgrep  2006-04-15 08:33:46.000000000 +0100
+++ bin/zgrep   2006-06-08 15:55:19.000000000 +0100
@@ -106,7 +106,24 @@
       else
        sed_script="s|^|${i}:|"
       fi
-      $grep $opt "$pat" | sed "$sed_script"
+      # Hack adapted from GPLed code at
+      # http://home.comcast.net/~j.p.h/cus-faq-2
+      # Has the same effect as the following two lines of bash:
+      #
+      # $grep $opt "$pat" | sed "$sed_script"
+      # exit ${PIPESTATUS[0]}
+      #
+      # Inside the `...`, fd4 goes to the pipe whose other end is read
+      # and passed to eval; fd1 is the normal standard output
+      # preserved the line before with exec 3>&1
+      exec 3>&1
+      eval `
+      exec 4>&1 >&3 3>&-
+      {
+       $grep $opt "$pat" 4>&-; echo "r=$?;" >&4
+      } | sed "$sed_script"
+      `
+      exit $r
     fi
   r=$?
   test $res -lt $r && res=$r
-----------cut here------------------

-- System Information:
Debian Release: testing/unstable
  APT prefers testing
  APT policy: (500, 'testing')
Architecture: i386 (i686)
Shell:  /bin/sh linked to /bin/bash
Kernel: Linux 2.6.15
Locale: LANG=en_GB.UTF-8, LC_CTYPE=en_GB.UTF-8 (charmap=UTF-8)

Versions of packages gzip depends on:
ii  debianutils                   2.16.1     Miscellaneous utilities specific t
ii  libc6                         2.3.6-13   GNU C Library: Shared libraries

gzip recommends no packages.

-- no debconf information


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to