Port to Solaris 11 /bin/sh

* m4/shell.m4 (AC_PROG_SHELL): Reject Solaris 11 /bin/sh.
A problem was reported by Nelson H. F. Beebe for OpenIndiana.
I reproduced a problem with different symptoms on Solaris 11.
Switching to Bash fixed it, and I hope this fixes it for
OpenIndiana too, since both problems appear to be shell-related.

diff --git a/m4/shell.m4 b/m4/shell.m4
index 3027aea46535ed225f317204f28aebb51dd416c5..e2f3fa90e08f508947fc6c34c742374fa775f95b 100644 (file)
--- a/m4/shell.m4
+++ b/m4/shell.m4
@@ -34,13 +34,21 @@ AC_DEFUN([AC_PROG_SHELL],
         # Test that $(...) works.
         test "$(expr 3 + 4)" -eq 7 || exit
       '
+      # Solaris 11 /bin/sh (AT&T Research) 93u 2011-02-08 has a file
+      # descriptor bug that breaks zgrep is are hard to test for
+      # directly.  At some point $(...) is reading a pipe from the
+      # subshell, but the pipe is also open in the parent shell, so
+      # the read deadlocks.  Prefer some other shell if available.
+      ac_maybe_solaris_sh_bug='test "${.sh.version}"'
+
       ac_cv_path_shell=no
 
       case $SHELL in
       /*)
         rm -f conftest.c
         if ("$SHELL" -c "$ac_command") 2>/dev/null; then
-          ac_cv_path_shell=$SHELL
+          "$SHELL" -c "$ac_maybe_solaris_sh_bug" 2>/dev/null ||
+            ac_cv_path_shell=$SHELL
         fi
       esac
 
@@ -52,6 +60,7 @@ AC_DEFUN([AC_PROG_SHELL],
         # PATH.  Also, loop through PATH first and then through
         # shells, since less-"nice" shells in /bin and /usr/bin are
         # more likely to be installed than "nicer" shells elsewhere.
+        ac_break_if_good_shell=:
         as_save_IFS=$IFS; IFS=:
         for as_dir in /bin /usr/bin $PATH
         do
@@ -61,13 +70,17 @@ AC_DEFUN([AC_PROG_SHELL],
             for ac_base in sh bash ksh sh5; do
               rm -f conftest.c
               if ("$as_dir/$ac_base" -c "$ac_command") 2>/dev/null; then
-                ac_cv_path_shell=$as_dir/$ac_base
-                break
+                if "$as_dir/$ac_base" -c "$ac_maybe_solaris_sh_bug" 2>/dev/null
+                then
+                  test "$ac_cv_path_shell" = no
+                else
+                  ac_break_if_good_shell=break
+                  :
+                fi && ac_cv_path_shell=$as_dir/$ac_base
+                $ac_break_if_good_shell
               fi
             done
-            case $ac_cv_path_shell in
-            /*) break
-            esac
+            $ac_break_if_good_shell
           esac
         done
         rm -f conftest.c
@@ -78,4 +91,7 @@ AC_DEFUN([AC_PROG_SHELL],
      SHELL=/bin/sh
      AC_MSG_WARN([using $SHELL, even though it does not conform to POSIX])
    fi
+   if "$SHELL" -c "$ac_maybe_solaris_sh_bug" 2>/dev/null; then
+     AC_MSG_WARN([using $SHELL, even though it may have file descriptor bugs])
+   fi
    AC_SUBST(SHELL)])