Clean up some bugs caught by preliminary dependency validation.

* lib/autoconf/headers.m4 (AC_DIR_HEADER): Don't invoke
AC_HEADER_DIRENT, since AC_FUNC_CLOSEDIR_VOID requires it.
* lib/m4sugar/m4sh.m4 (_AS_DETECT_BETTER_SHELL)
(_AS_SHELL_SANITIZE): Fix quoting bugs.

Signed-off-by: Eric Blake <ebb9@byu.net>

diff --git a/ChangeLog b/ChangeLog
index 7029b84c538982b4ed2286fe2ac6e3cafeb920be..60235986cbbd42a279094a52d447f48291379c12 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2009-01-20  Eric Blake  <ebb9@byu.net>
+
+       Clean up some bugs caught by preliminary dependency validation.
+       * lib/autoconf/headers.m4 (AC_DIR_HEADER): Don't invoke
+       AC_HEADER_DIRENT, since AC_FUNC_CLOSEDIR_VOID requires it.
+       * lib/m4sugar/m4sh.m4 (_AS_DETECT_BETTER_SHELL)
+       (_AS_SHELL_SANITIZE): Fix quoting bugs.
+
 2009-01-19  Eric Blake  <ebb9@byu.net>
 
        Improve AC_REQUIRE documentation.

diff --git a/lib/autoconf/headers.m4 b/lib/autoconf/headers.m4
index 0bf4f60afd11150e82547c4b402836023e73036d..47468f088a2fa7edf9a152c0932669da0f246e06 100644 (file)
--- a/lib/autoconf/headers.m4
+++ b/lib/autoconf/headers.m4
@@ -1,8 +1,8 @@
 # This file is part of Autoconf.                       -*- Autoconf -*-
 # Checking for headers.
 #
-# Copyright (C) 1988, 1999, 2000, 2001, 2002, 2003, 2004, 2006, 2008
-# Free Software Foundation, Inc.
+# Copyright (C) 1988, 1999, 2000, 2001, 2002, 2003, 2004, 2006, 2008,
+# 2009 Free Software Foundation, Inc.
 #
 # This program is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
@@ -439,11 +439,11 @@ AC_DEFUN([AC_HEADER_ASSERT],
     [ac_enable_assert=$enableval
      AS_IF(dnl
       [test "x$enableval" = xno],
-        [AC_DEFINE([NDEBUG], [1],
-          [Define to 1 if assertions should be disabled.])],
+       [AC_DEFINE([NDEBUG], [1],
+         [Define to 1 if assertions should be disabled.])],
       [test "x$enableval" != xyes],
-        [AC_MSG_WARN([invalid argument supplied to --enable-assert])
-        ac_enable_assert=yes])],
+       [AC_MSG_WARN([invalid argument supplied to --enable-assert])
+       ac_enable_assert=yes])],
     [ac_enable_assert=yes])
   AC_MSG_RESULT([$ac_enable_assert])
 ])
@@ -887,8 +887,7 @@ use HAVE_STRING_H and HAVE_MEMORY_H, not NEED_MEMORY_H.])
 # defines a different set of C preprocessor macros to indicate which
 # header file is found.
 AU_DEFUN([AC_DIR_HEADER],
-[AC_HEADER_DIRENT
-AC_FUNC_CLOSEDIR_VOID
+[AC_FUNC_CLOSEDIR_VOID
 test ac_cv_header_dirent_dirent_h &&
   AC_DEFINE([DIRENT], 1, [Same as `HAVE_DIRENT_H', don't depend on me.])
 test ac_cv_header_dirent_sys_ndir_h &&

diff --git a/lib/m4sugar/m4sh.m4 b/lib/m4sugar/m4sh.m4
index 0c4c37fb43f5114abd8fac6a7dbb489494c259b9..2e17a9de7f5f178a383bae025c5ef3da755f59c7 100644 (file)
--- a/lib/m4sugar/m4sh.m4
+++ b/lib/m4sugar/m4sh.m4
@@ -217,7 +217,7 @@ m4_define([_AS_DETECT_SUGGESTED_PRUNE],
 m4_defun([_AS_DETECT_BETTER_SHELL],
 dnl Remove any tests from suggested that are also required
 [m4_set_map([_AS_DETECT_SUGGESTED_BODY], [_AS_DETECT_SUGGESTED_PRUNE])]dnl
-[m4_pushdef([AS_EXIT], [exit m4_default([$1], 1)])]dnl
+[m4_pushdef([AS_EXIT], [exit m4_default(]m4_dquote([$][1])[, 1)])]dnl
 [if test "x$CONFIG_SHELL" = x; then
   as_bourne_compatible="AS_ESCAPE(_m4_expand([_AS_BOURNE_COMPATIBLE]))"
   _AS_DETECT_EXPAND([as_required], [_AS_DETECT_REQUIRED_BODY])
@@ -429,7 +429,7 @@ test x$exitcode = x0[]])# _AS_SHELL_FN_WORK
 # it is executed prior to shell function definitions, hence the
 # temporary redefinition of AS_EXIT.
 m4_defun([_AS_SHELL_SANITIZE],
-[m4_pushdef([AS_EXIT], [exit m4_default([$1], 1)])]dnl
+[m4_pushdef([AS_EXIT], [exit m4_default(]m4_dquote([$][1])[, 1)])]dnl
 [m4_text_box([M4sh Initialization.])
 
 AS_BOURNE_COMPATIBLE