bad m4 quotation? (was: echo "enhancement" leads to confused legacy script tools...)

[Ralf Wildenhues]

* Ralf Wildenhues wrote on Tue, Mar 21, 2006 at 05:32:44PM CET:
> 
> I will followup (on bug-autoconf only) with a bug in CVS Autoconf that I
> just discovered, that led me to thinking the AX_PREFIX_CONFIG_H macro
> was broken..

Hmm.  This works:

AC_INIT(a,1,b)
AC_DEFUN([FOO],
[AC_CONFIG_FILES(m4_default([$1], [foobar]))])
FOO
FOO([baz])
AC_OUTPUT

But this fails:

AC_INIT(a,1,b)
AC_DEFUN([FOO],
[AC_CONFIG_FILES([m4_default([$1], [foobar])])])
FOO
FOO([baz])
AC_OUTPUT

Is that a bug?  In above code or in Autoconf?

What about the usage of `ifelse' in AX_PREFIX_CONFIG_H?
Would that also need to be outside []?
Cf. this patch against
http://autoconf-archive.cryp.to/ax_prefix_config_h.html

Thanks for enlightening me.

Cheers,
Ralf


--- ax_prefix_config_h.m4       2006-03-21 17:46:26.000000000 +0100
+++ ax_prefix_config_h.m4       2006-03-21 17:46:43.000000000 +0100
@@ -88,7 +88,7 @@
 dnl @license GPLWithACException
 
 AC_DEFUN([AX_PREFIX_CONFIG_H],[AC_REQUIRE([AC_CONFIG_HEADER])
-AC_CONFIG_COMMANDS([ifelse($1,,$PACKAGE-config.h,$1)],[dnl
+AC_CONFIG_COMMANDS(ifelse($1,,$PACKAGE-config.h,$1),[dnl
 AS_VAR_PUSHDEF([_OUT],[ac_prefix_conf_OUT])dnl
 AS_VAR_PUSHDEF([_DEF],[ac_prefix_conf_DEF])dnl
 AS_VAR_PUSHDEF([_PKG],[ac_prefix_conf_PKG])dnl