from Jim meyering.

1 files changed, 35 insertions, 35 deletions

diff --git a/m4/regex.m4 b/m4/regex.m4
index 0456c74e..3047d4d4 100644
--- a/m4/regex.m4
+++ b/m4/regex.m4
@@ -1,54 +1,53 @@
-#serial 4
+#serial 5
 
 dnl Initially derived from code in GNU grep.
 dnl Mostly written by Jim Meyering.
 
+dnl Usage: jm_INCLUDED_REGEX([lib/regex.c])
+dnl
 AC_DEFUN(jm_INCLUDED_REGEX,
   [
     dnl Even packages that don't use regex.c can use this macro.
     dnl Of course, for them it doesn't do anything.
 
-    # Normally we should skip the test for GLIBC but a bug
-    # in the older regex will not be detected so test for everyone.
-    # The failing regular expression is from `Spencer ere test
-    # #75' in grep-2.2f.
-    dnl AC_REQUIRE([AM_GLIBC])
-
+    # Assume we'll default to using the included regex.c.
     ac_use_included_regex=yes
 
-    # Without this run-test, on older glibc2 systems we'd end up
-    # using the buggy system regex.
+    # However, if the system regex support is good enough that it passes the
+    # the following run test, then default to *not* using the included regex.c.
+    # If cross compiling, assume the test would fail and use the included
+    # regex.c.  The failing regular expression is from `Spencer ere test #75'
+    # in grep-2.3.
     AC_CACHE_CHECK([for working re_compile_pattern],
-                   jm_cv_func_working_re_compile_pattern,
-	AC_TRY_RUN(
-	  changequote(<<, >>)dnl
-	  <<
+		   jm_cv_func_working_re_compile_pattern,
+      AC_TRY_RUN(
+	changequote(<<, >>)dnl
+	<<
 #include <stdio.h>
 #include <regex.h>
-	    int
-	    main ()
-	    {
-	      static struct re_pattern_buffer regex;
-	      const char *s;
-	      re_set_syntax (RE_SYNTAX_POSIX_EGREP);
-	      /* Add this third left square bracket, XX, to balance the
-		 three right ones below.  Otherwise autoconf-2.14 chokes.  */
-	      s = re_compile_pattern ("a[[:XX:]]b\n", 9, &regex);
-	      /* This should fail with _Invalid character class name_ error.  */
-	      exit (s ? 0 : 1);
-	    }
-	  >>,
-	  changequote([, ])dnl
+	  int
+	  main ()
+	  {
+	    static struct re_pattern_buffer regex;
+	    const char *s;
+	    re_set_syntax (RE_SYNTAX_POSIX_EGREP);
+	    /* Add this third left square bracket, [, to balance the
+	       three right ones below.  Otherwise autoconf-2.14 chokes.  */
+	    s = re_compile_pattern ("a[[:]:]]b\n", 9, &regex);
+	    /* This should fail with _Invalid character class name_ error.  */
+	    exit (s ? 0 : 1);
+	  }
+	>>,
+	changequote([, ])dnl
 
-	 jm_cv_func_working_re_compile_pattern=yes,
-	 jm_cv_func_working_re_compile_pattern=no,
-	 dnl When crosscompiling, assume it's broken.
-	 jm_cv_func_working_re_compile_pattern=no))
-    if test "$jm_cv_func_working_re_compile_pattern" = yes; then
+	       jm_cv_func_working_re_compile_pattern=yes,
+	       jm_cv_func_working_re_compile_pattern=no,
+	       dnl When crosscompiling, assume it's broken.
+	       jm_cv_func_working_re_compile_pattern=no))
+    if test $jm_cv_func_working_re_compile_pattern = yes; then
       ac_use_included_regex=no
-     fi
+    fi
 
-    AC_SUBST(LIBOJS)dnl
     test -n "$1" || AC_MSG_ERROR([missing argument])
     syscmd([test -f $1])
     ifelse(sysval, 0,
@@ -61,7 +60,8 @@ AC_DEFUN(jm_INCLUDED_REGEX,
 		    jm_with_regex=$withval,
 		    jm_with_regex=$ac_use_included_regex)
 	if test "$jm_with_regex" = yes; then
-	  LIBOBJS="$LIBOBJS regex.${ac_objext}"
+	  AC_SUBST(LIBOBJS)
+	  LIBOBJS="$LIBOBJS regex.$ac_objext"
 	fi
       ],
     )