Hello,
On Fri, May 19, 2006 at 05:04:00PM +0200, Akim Demaille wrote:
> I have this package at hand, say FooBar, which installs a prefixed
> form of config.h. Its own macros are, of course, named FB_*, which is
> m4_pattern_forbidden. But then I have to explicitly m4_pattern_allow
> all my FB_PACKAGE_VERSION etc.
>
> I suggest that the AC_DEFINE family explicitly allow its $1.
this sounds good. (Note that AC_SUBST already allows its parameter.)
Proposed patch attached.
I think it is so safe that it might be installed right now, but not
without Ralf's approval. Ralf?
Have a nice day,
Stepan Kasal
2006-05-22 Stepan Kasal <[EMAIL PROTECTED]>
* lib/autoconf/general.m4 (AC_DEFINE_TRACE_LITERAL): Pass
the parameter to m4_pattern_allow.
Index: lib/autoconf/general.m4
===================================================================
RCS file: /cvsroot/autoconf/autoconf/lib/autoconf/general.m4,v
retrieving revision 1.918
diff -u -r1.918 general.m4
--- lib/autoconf/general.m4 21 May 2006 00:19:42 -0000 1.918
+++ lib/autoconf/general.m4 22 May 2006 12:45:28 -0000
@@ -1940,9 +1940,9 @@
# AC_DEFINE_TRACE_LITERAL(LITERAL-CPP-SYMBOL)
# -------------------------------------------
-# This macro is useless, it is used only with --trace to collect the
-# list of *literals* CPP values passed to AC_DEFINE/AC_DEFINE_UNQUOTED.
-m4_define([AC_DEFINE_TRACE_LITERAL])
+# Used by --trace to collect the list of AC_DEFINEd macros.
+m4_define([AC_DEFINE_TRACE_LITERAL],
+[m4_pattern_allow([^$1$])])
# AC_DEFINE_TRACE(CPP-SYMBOL)
