changeset 12893:1e998663ea99

lseek: avoid bash 3.2 broken pipe bug Avoid a spurious message caused by SIGPIPE handling, as well as avoiding bash 3.2 messages even when SIGPIPE is not handled. * m4/lseek.m4 (gl_FUNC_LSEEK): Drain pipe, to avoid spurious warning from bash 3.2. Reported by Ben Pfaff, with analysis from Bruno Haible. Signed-off-by: Eric Blake <eblake@redhat.com>
author Eric Blake <eblake@redhat.com>
date Mon, 22 Feb 2010 15:15:56 -0700
parents 81072b472b9d
children e10b357eec3a
files ChangeLog m4/lseek.m4
diffstat 2 files changed, 8 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,10 @@
 2010-02-22  Eric Blake  <eblake@redhat.com>
 
+	lseek: avoid bash 3.2 broken pipe bug
+	* m4/lseek.m4 (gl_FUNC_LSEEK): Drain pipe, to avoid spurious
+	warning from bash 3.2.
+	Reported by Ben Pfaff, with analysis from Bruno Haible.
+
 	bootstrap: support non-FSF copyright holder
 	* build-aux/bootstrap (COPYRIGHT_HOLDER, with_gettext): Allow
 	bootstrap.conf override of COPYRIGHT_HOLDER.
--- a/m4/lseek.m4
+++ b/m4/lseek.m4
@@ -1,4 +1,4 @@
-# lseek.m4 serial 4
+# lseek.m4 serial 5
 dnl Copyright (C) 2007, 2009, 2010 Free Software Foundation, Inc.
 dnl This file is free software; the Free Software Foundation
 dnl gives unlimited permission to copy and/or distribute it,
@@ -21,7 +21,8 @@
 }],
          [if test -s conftest$ac_exeext \
              && ./conftest$ac_exeext < conftest.$ac_ext \
-             && { echo hi | ./conftest$ac_exeext; test $? = 1; }; then
+             && test 1 = "`echo hi \
+               | { ./conftest$ac_exeext; echo $?; cat >/dev/null; }`"; then
             gl_cv_func_lseek_pipe=yes
           else
             gl_cv_func_lseek_pipe=no