]> git.gag.com Git - debian/gzip/commitdiff
* tests/zgrep-signal: Use perl instead of a nonportable shell trap.
authorPaul Eggert <eggert@cs.ucla.edu>
Wed, 2 Nov 2011 21:27:33 +0000 (14:27 -0700)
committerPaul Eggert <eggert@cs.ucla.edu>
Wed, 2 Nov 2011 21:27:53 +0000 (14:27 -0700)
Problem reported by Eric Blake in
<http://lists.gnu.org/archive/html/bug-gzip/2011-11/msg00005.html>.

tests/zgrep-signal

index 3c5b6966d18848bc2d728f8d936b76a103d4262b..e19293a824cf715330037a9e9668e8759069c065 100644 (file)
 
 echo a | gzip -c > f.gz || framework_failure_
 
-if (trap - PIPE) >/dev/null 2>&1; then
-  trap - PIPE
-else
-  trap 13
-fi
-st=$(exec 3>&1; (cat f.gz f.gz; echo $? >&3) | :)
+test "x$PERL" = x && PERL=perl
+("$PERL" -e 'use warnings') >/dev/null 2>&1 || skip_ "no suitable perl found"
+
+exec_with_SIGPIPE_SIG_DFL () {
+  program=${1?}
+  shift
+  args=
+  for arg; do
+    args="$args, '$arg'"
+  done
+  "$PERL" -e "\$SIG{PIPE} = 'DEFAULT'; exec '$program'$args"
+}
+
+write_to_dangling_pipe () {
+  exec 3>&1
+  (
+    exec_with_SIGPIPE_SIG_DFL "$@"
+    echo $? >&3
+  ) | : || framework_failure_
+}
+
+st=$(write_to_dangling_pipe cat f.gz f.gz)
 test "$st" = 141 || framework_failure_ 'signal handling busted on this host'
 
 fail=0
 
-st=$(exec 3>&1; (zgrep a f.gz f.gz; echo $? >&3) | :) || framework_failure_
+st=$(write_to_dangling_pipe zgrep a f.gz f.gz)
 
 test "$st" = 141 || fail=1