From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from localhost (localhost [127.0.0.1]) by arlo.cworth.org (Postfix) with ESMTP id 1A9666DE109B for ; Sun, 9 Jun 2019 10:46:49 -0700 (PDT) X-Virus-Scanned: Debian amavisd-new at cworth.org X-Spam-Flag: NO X-Spam-Score: -0.046 X-Spam-Level: X-Spam-Status: No, score=-0.046 tagged_above=-999 required=5 tests=[AWL=-0.045, SPF_PASS=-0.001] autolearn=disabled Received: from arlo.cworth.org ([127.0.0.1]) by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id qwhtPEONGJjg for ; Sun, 9 Jun 2019 10:46:48 -0700 (PDT) Received: from fethera.tethera.net (fethera.tethera.net [198.245.60.197]) by arlo.cworth.org (Postfix) with ESMTPS id 146176DE1029 for ; Sun, 9 Jun 2019 10:46:47 -0700 (PDT) Received: from remotemail by fethera.tethera.net with local (Exim 4.89) (envelope-from ) id 1ha1u1-0000pK-Da; Sun, 09 Jun 2019 13:46:45 -0400 Received: (nullmailer pid 32369 invoked by uid 1000); Sun, 09 Jun 2019 17:47:07 -0000 From: David Bremner To: Ralph Seichter , notmuch@notmuchmail.org Subject: Re: Bug: Attempt to install notmuch-emacs.info during --without-emacs build In-Reply-To: <87k1duiw05.fsf@ra.horus-it.com> References: <87k1duiw05.fsf@ra.horus-it.com> Date: Sun, 09 Jun 2019 14:47:07 -0300 Message-ID: <87d0jmy7l0.fsf@tethera.net> MIME-Version: 1.0 Content-Type: text/plain X-BeenThere: notmuch@notmuchmail.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: "Use and development of the notmuch mail system." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 09 Jun 2019 17:46:49 -0000 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)