Ralf Wildenhues <Ralf.Wildenhues <at> gmx.de> writes:

> 
> So, better would be
>   m4_pattern_forbid([^gl_[A-Z]])
>   m4_pattern_allow([^gl_ES])
> 
> the first of which is similarly used in coreutils/m4/prereq.m4
> already (which in turn was part of Gnulib for a while).

Agreed.  Checking in as follows:

2006-08-10  Eric Blake  <[EMAIL PROTECTED]>
        and Ralf Wildenhues <[EMAIL PROTECTED]>

        * gnulib-tool (func_import): Detect unexpanded macros in gnulib
        namespace.

Index: gnulib-tool
===================================================================
RCS file: /sources/gnulib/gnulib/gnulib-tool,v
retrieving revision 1.134
diff -u -r1.134 gnulib-tool
--- gnulib-tool 8 Aug 2006 11:35:22 -0000       1.134
+++ gnulib-tool 10 Aug 2006 14:17:01 -0000
@@ -1526,6 +1526,8 @@
     echo "# any checks for libraries, header files, types and library 
functions."
     echo "AC_DEFUN([${macro_prefix}_EARLY],"
     echo "["
+    echo "  m4_pattern_forbid([^gl_[A-Z]])dnl the gnulib macro namespace"
+    echo "  m4_pattern_allow([^gl_ES$])dnl a valid locale name"
     echo "  AC_REQUIRE([AC_PROG_RANLIB])"
     if grep AC_GNU_SOURCE "$destdir"/$m4base/*.m4 >/dev/null 2>/dev/null; then
       echo "  AC_REQUIRE([AC_GNU_SOURCE])"





Reply via email to