Bug: Attempt to install notmuch-emacs.info during --without-emacs build

[David Bremner]

Ralph Seichter <abbot at monksofcool.net> 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)