On 18/01/2026 05:29, Collin Funk wrote:
Hi Pádraig,
After pushing my last patch, I noticed that the html post processing
depends on GNU Make being used since MAKEINFO is defined to the
makeinfo-wrapper.sh script in cfg.mk. That file is only included in
GNUMakefile.
What do you think about this patch turning the script into a file to
be created by AC_CONFIG_FILES? I don't think it can be done any other
way, e.g., an environment variable, since MAKEINFO will be defined to
a value with spaces, "${SHELL} build-aux/missing makeinfo".
I see the web-manual make target in gnulib/top/maint.mk
which documents cfg.mk is required.
I.e., GNU make is required for that target.
But you're right since there is a separate "html" target,
which would not have the right anchors if built with non gnu make.
diff --git a/doc/local.mk b/doc/local.mk
index 04ce909e9..6f80cb189 100644
--- a/doc/local.mk
+++ b/doc/local.mk
@@ -25,9 +25,11 @@ doc_coreutils_TEXINFOS = \
doc/fdl.texi \
doc/sort-version.texi
+MAKEINFO = $(abs_top_srcdir)/build-aux/makeinfo-wrapper.sh
I think that should be ${abs_top_builddir} to support VPATH builds.
thanks!
Padraig
