Bug: Attempt to install notmuch-emacs.info during --without-emacs build
MacPorts CI builds report the following [1]: DEBUG: system: cd "/opt/local/var/macports/build/_Users_vsts_agent_2.152.1_work_1_s_mail_notmuch/notmuch/work/notmuch-0.29" && ./configure --prefix=/opt/local --with-docs --without-emacs --without-ruby [...] Checking if emacs (>= 24) is available... No (so will not byte-compile emacs code) [...] /opt/local/var/macports/build/_Users_vsts_agent_2.152.1_work_1_s_mail_notmuch/notmuch/work/notmuch-0.29/doc/index.rst:7: WARNING: toctree contains reference to excluded document 'notmuch-emacs' WARNING: "texinfo_documents" config value references unknown document notmuch-emacs [...] install -m0644 doc/_build/texinfo/notmuch-address.info doc/_build/texinfo/notmuch-compact.info doc/_build/texinfo/notmuch-config.info doc/_build/texinfo/notmuch-count.info doc/_build/texinfo/notmuch-dump.info doc/_build/texinfo/notmuch-emacs-mua.info doc/_build/texinfo/notmuch-insert.info doc/_build/texinfo/notmuch-new.info doc/_build/texinfo/notmuch-reindex.info doc/_build/texinfo/notmuch-reply.info doc/_build/texinfo/notmuch-restore.info doc/_build/texinfo/notmuch-search.info doc/_build/texinfo/notmuch-show.info doc/_build/texinfo/notmuch-tag.info doc/_build/texinfo/notmuch.info doc/_build/texinfo/notmuch-hooks.info doc/_build/texinfo/notmuch-properties.info doc/_build/texinfo/notmuch-search-terms.info doc/_build/texinfo/notmuch-emacs.info /opt/local/var/macports/build/_Users_vsts_agent_2.152.1_work_1_s_mail_notmuch/notmuch/work/destroot/opt/local/share/info install: doc/_build/texinfo/notmuch-emacs.info: No such file or directory make: *** [install-info] Error 71 Why is the build trying to install texinfo/notmuch-emacs.info even though --without-emacs was specified during configuration and Emacs was not found during the sanity checks? [1] Full build log available at https://paste.z0k.xyz/8352cabdee6f.txt -Ralph ___ notmuch mailing list notmuch@notmuchmail.org https://notmuchmail.org/mailman/listinfo/notmuch
Re: Bug: Attempt to install notmuch-emacs.info during --without-emacs build
Ralph Seichter writes:
>
> Why is the build trying to install texinfo/notmuch-emacs.info even
> though --without-emacs was specified during configuration and Emacs was
> not found during the sanity checks?
Agreed, that looks like a bug. Apprently the combination of having
makeinfo but not emacs is not well tested.
The following seems to fix the problem, but I haven't tested it extensively.
diff --git a/doc/Makefile.local b/doc/Makefile.local
index 719172fe..55ff4a64 100644
--- a/doc/Makefile.local
+++ b/doc/Makefile.local
@@ -28,7 +28,10 @@ MAN_GZIP_FILES := $(addsuffix .gz,${MAN_ROFF_FILES})
MAN1_TEXI := $(patsubst
$(srcdir)/doc/man1/%.rst,$(DOCBUILDDIR)/texinfo/%.texi,$(MAN1_RST))
MAN5_TEXI := $(patsubst
$(srcdir)/doc/man5/%.rst,$(DOCBUILDDIR)/texinfo/%.texi,$(MAN5_RST))
MAN7_TEXI := $(patsubst
$(srcdir)/doc/man7/%.rst,$(DOCBUILDDIR)/texinfo/%.texi,$(MAN7_RST))
-INFO_TEXI_FILES := $(MAN1_TEXI) $(MAN5_TEXI) $(MAN7_TEXI)
$(DOCBUILDDIR)/texinfo/notmuch-emacs.texi
+INFO_TEXI_FILES := $(MAN1_TEXI) $(MAN5_TEXI) $(MAN7_TEXI)
+ifeq ($(WITH_EMACS),1)
+ $(INFO_TEXI_FILES):= $(INFO_TEXI_FILES)
$(DOCBUILDDIR)/texinfo/notmuch-emacs.texi
+endif
INFO_INFO_FILES := $(INFO_TEXI_FILES:.texi=.info)
___
notmuch mailing list
notmuch@notmuchmail.org
https://notmuchmail.org/mailman/listinfo/notmuch
Re: Bug: Attempt to install notmuch-emacs.info during --without-emacs build
* David Bremner: > The following seems to fix the problem, but I haven't tested it > extensively. As shown in https://monksofcool.net/tmp/6bmjUFL.log , your suggestion results in notmuch-emacs.info being generated but unfortunately not installed when Notmuch is built --with-emacs. Obviously that's better than a build failure for --without-emacs, but it is not quite ideal either. ;-) -Ralph ___ notmuch mailing list notmuch@notmuchmail.org https://notmuchmail.org/mailman/listinfo/notmuch
Re: Bug: Attempt to install notmuch-emacs.info during --without-emacs build
Ralph Seichter writes:
> * David Bremner:
>
>> The following seems to fix the problem, but I haven't tested it
>> extensively.
>
> As shown in https://monksofcool.net/tmp/6bmjUFL.log , your suggestion
> results in notmuch-emacs.info being generated but unfortunately not
> installed when Notmuch is built --with-emacs. Obviously that's better
> than a build failure for --without-emacs, but it is not quite ideal
> either. ;-)
>
Attempt #2; we need to delay expansion of INFO_TEXI_FILES until
WITH_EMACS is defined.
diff --git a/doc/Makefile.local b/doc/Makefile.local
index 719172fe..5b99dd02 100644
--- a/doc/Makefile.local
+++ b/doc/Makefile.local
@@ -28,7 +28,9 @@ MAN_GZIP_FILES := $(addsuffix .gz,${MAN_ROFF_FILES})
MAN1_TEXI := $(patsubst
$(srcdir)/doc/man1/%.rst,$(DOCBUILDDIR)/texinfo/%.texi,$(MAN1_RST))
MAN5_TEXI := $(patsubst
$(srcdir)/doc/man5/%.rst,$(DOCBUILDDIR)/texinfo/%.texi,$(MAN5_RST))
MAN7_TEXI := $(patsubst
$(srcdir)/doc/man7/%.rst,$(DOCBUILDDIR)/texinfo/%.texi,$(MAN7_RST))
-INFO_TEXI_FILES := $(MAN1_TEXI) $(MAN5_TEXI) $(MAN7_TEXI)
$(DOCBUILDDIR)/texinfo/notmuch-emacs.texi
+INFO_TEXI_FILES = $(MAN1_TEXI) $(MAN5_TEXI) $(MAN7_TEXI) \
+ $(if $(filter 1,$(WITH_EMACS)),
$(DOCBUILDDIR)/texinfo/notmuch-emacs.texi,)
+
INFO_INFO_FILES := $(INFO_TEXI_FILES:.texi=.info)
.PHONY: sphinx-html sphinx-texinfo sphinx-info
___
notmuch mailing list
notmuch@notmuchmail.org
https://notmuchmail.org/mailman/listinfo/notmuch
Re: Bug: Attempt to install notmuch-emacs.info during --without-emacs build
* David Bremner: > Attempt #2; we need to delay expansion of INFO_TEXI_FILES until > WITH_EMACS is defined. Works nicely for my local builds. Now I need to wait for the CI builds to run. -Ralph ___ notmuch mailing list notmuch@notmuchmail.org https://notmuchmail.org/mailman/listinfo/notmuch
Re: Bug: Attempt to install notmuch-emacs.info during --without-emacs build
MacPorts CI builds were successful as well. Thank you for the prompt fix, David. -Ralph ___ notmuch mailing list notmuch@notmuchmail.org https://notmuchmail.org/mailman/listinfo/notmuch
