m4_foreach_w(...AC_DEFINE...) expansion change in 2.72e?

Karl Berry <[email protected]>
Newsgroups gmane.comp.sysutils.autoconf.bugs
Message-ID <[email protected]>
2.72e creates a different autoconf.h.in and configure in TeX Live's
texk/lcdf-typetools
(https://github.com/TeX-Live/texlive-source/tree/trunk/texk/lcdf-typetools):

--- autoconf.h.in	(revision 69180)
+++ autoconf.h.in	(working copy)
-/* Define to run cfftot1 from otftotfm. */
-#undef HAVE_AUTO_CFFTOT1
+/* Define to run cfftot1 t1dotlessj ttftotype42 updmap from otftotfm. */
+#undef HAVE_AUTO_CFFTOT1_T1DOTLESSJ_TTFTOTYPE42_UPDMAP
-/* Define to run t1dotlessj from otftotfm. */
-#undef HAVE_AUTO_T1DOTLESSJ
-
-/* Define to run ttftotype42 from otftotfm. */
-#undef HAVE_AUTO_TTFTOTYPE42
-
-/* Define to run updmap from otftotfm. */
-#undef HAVE_AUTO_UPDMAP
-

In other words, what was four separate HAVE_AUTO_* macros have become one.

The configure.ac code that generates this runs through those four values
using m4_foreach_w:

---
dnl the four values:
m4_define([kpse_otftotfm_auto_opts], [cfftot1 t1dotlessj ttftotype42 updmap])[]dnl
dnl define the enable/disable options for each:
AC_FOREACH([Kpse_Opt], kpse_otftotfm_auto_opts,
           [AC_ARG_ENABLE(Kpse_Opt,
                          AS_HELP_STRING([--disable-auto-]Kpse_Opt,
                                         [disable running ]Kpse_Opt[ from otftotfm],
                                         kpse_indent_26))])
..
dnl ac_define each of them accordingly.
m4_foreach_w([Kpse_Opt],[kpse_otftotfm_auto_opts],[AS_IF([test "x$enable_auto_]Kpse_Opt[" != xno],
                  [AC_DEFINE([HAVE_AUTO_]AS_TR_CPP(Kpse_Opt), 1,
                             [Define to run ]Kpse_Opt[ from otftotfm.])])
---

Quoting the [kpse_otftotfm_auto_opts] vs. plain kpse_otftotfm_auto_opts
makes no difference.

I guess I could unroll the loop into separate AC_DEFINE calls instead of
doing it in a loop, but is there something in there that is written
incorrectly? I'm not sure, so I thought I'd report it. Autoconf did not
report any warnings about it, as far as I could see. --thanks, karl.
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.