* m4/shell.m4 (AC_PROG_SHELL): Check for old-fashioned shells that

don't allow "(" in front of case labels.  Problem reported by midair
in <http://lists.gnu.org/archive/html/bug-gzip/2007-04/msg00004.html>.
Also, check for buggy shells that require ";;" before "esac".
This commit is contained in:
Paul Eggert 2007-04-29 05:24:26 +00:00
parent 0291f786ec
commit 79c3261c8b
2 changed files with 14 additions and 7 deletions

View File

@ -1,3 +1,10 @@
2007-04-28 Paul Eggert <eggert@cs.ucla.edu>
* m4/shell.m4 (AC_PROG_SHELL): Check for old-fashioned shells that
don't allow "(" in front of case labels. Problem reported by midair
in <http://lists.gnu.org/archive/html/bug-gzip/2007-04/msg00004.html>.
Also, check for buggy shells that require ";;" before "esac".
2007-04-13 Paul Eggert <eggert@cs.ucla.edu>
* NEWS: Version 1.3.12 released.

View File

@ -37,15 +37,15 @@ AC_DEFUN([AC_PROG_SHELL],
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
fi;;
fi
esac
case $ac_cv_path_shell in
no)
(no)
# Prefer shells that are more likely to be installed in the
# same place on all hosts of this platform. Therefore, prefer
# shells in /bin and /usr/bin to shells in the installer's
@ -57,7 +57,7 @@ AC_DEFUN([AC_PROG_SHELL],
do
IFS=$as_save_IFS
case $as_dir in
/*)
(/*)
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
@ -66,11 +66,11 @@ AC_DEFUN([AC_PROG_SHELL],
fi
done
case $ac_cv_path_shell in
/*) break;;
esac;;
(/*) break
esac
esac
done
rm -f conftest.c;;
rm -f conftest.c
esac])
AC_MSG_RESULT($ac_cv_path_shell)
SHELL=$ac_cv_path_shell