From mboxrd@z Thu Jan  1 00:00:00 1970
From: Mathieu Lirzin <mthl@gnu.org>
Subject: Re: [PATCH 1/2] build: Use an up to date copy of texinfo.tex
Date: Fri, 24 Feb 2017 21:54:57 +0100
Message-ID: <87mvdbtila.fsf@gnu.org>
References: <1487765072-32603-1-git-send-email-jmd@gnu.org>
	<1487765072-32603-2-git-send-email-jmd@gnu.org>
	<87r32ntmqn.fsf@gnu.org> <20170224201253.GA18175@jocasta.intra>
Mime-Version: 1.0
Content-Type: text/plain
Return-path: <guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org>
Received: from eggs.gnu.org ([2001:4830:134:3::10]:59594)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <mthl@gnu.org>) id 1chMtK-0003Qh-Pq
	for guix-devel@gnu.org; Fri, 24 Feb 2017 15:55:04 -0500
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <mthl@gnu.org>) id 1chMtJ-0007pd-90
	for guix-devel@gnu.org; Fri, 24 Feb 2017 15:55:02 -0500
In-Reply-To: <20170224201253.GA18175@jocasta.intra> (John Darrington's message
	of "Fri, 24 Feb 2017 21:12:53 +0100")
List-Id: "Development of GNU Guix and the GNU System distribution."
	<guix-devel.gnu.org>
List-Unsubscribe: <https://lists.gnu.org/mailman/options/guix-devel>,
	<mailto:guix-devel-request@gnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/archive/html/guix-devel/>
List-Post: <mailto:guix-devel@gnu.org>
List-Help: <mailto:guix-devel-request@gnu.org?subject=help>
List-Subscribe: <https://lists.gnu.org/mailman/listinfo/guix-devel>,
	<mailto:guix-devel-request@gnu.org?subject=subscribe>
Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org
Sender: "Guix-devel" <guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org>
To: John Darrington <john@darrington.wattle.id.au>
Cc: guix-devel@gnu.org, John Darrington <jmd@gnu.org>

John Darrington <john@darrington.wattle.id.au> writes:

> On Fri, Feb 24, 2017 at 08:25:20PM +0100, Mathieu Lirzin wrote:
>      
>      John Darrington <jmd@gnu.org> writes:
>      
>      > * build-aux/texinfo.tex,ref: New file, copied from texlive-minimal
>      > * bootstrap: Use it, if newer than the texinfo.tex from automake.
>      > ---
>      >  bootstrap                 |    11 +-
>      >  build-aux/texinfo.tex,ref | 11562 ++++++++++++++++++++++++++++++++++++++++++++
>      >  2 files changed, 11572 insertions(+), 1 deletion(-)
>      >  create mode 100644 build-aux/texinfo.tex,ref
>      >
>      
>      Do you know why building Guix manual fails to build with older
>      "texinfo.tex"?
>
>      If this related to some special texinfo syntax used in Guix manual?
>
> Yes.  It's the use of @inlinefmtifelse command. (perhaps other commands too).

There is only one instance of this command which is in "doc/guix.texi":

  If you are instead planning to encrypt the root partition, you can use
  the Cryptsetup/LUKS utilities to do that (see @inlinefmtifelse{html,
  @uref{https://linux.die.net/man/8/cryptsetup, @code{man cryptsetup}},
  @code{man cryptsetup}} for more information.)

I have tested that without it, 'make pdf' succeeds.  IMHO the manual
would be fine without this feature.  As a consequence --to avoid adding
complexity to the build process-- I would be in favour of not using
@inlinefmtifelse until Automake distributes a compatible version in its
current version.

>      What about checking in texinfo.tex and removing the -f (--force) option
>      when invoking 'autoreconf'?  This would ensure that texinfo.tex is not
>      overwritten.
>
>
> I guess it might work.  But I haven't tried it.

Thank you for the debugging information.  :)

-- 
Mathieu Lirzin
GPG: F2A3 8D7E EB2B 6640 5761  070D 0ADE E100 9460 4D37