From: Mathieu Lirzin <mthl@gnu.org>
To: John Darrington <jmd@gnu.org>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH 1/2] build: Use an up to date copy of texinfo.tex
Date: Fri, 24 Feb 2017 20:25:20 +0100 [thread overview]
Message-ID: <87r32ntmqn.fsf@gnu.org> (raw)
In-Reply-To: <1487765072-32603-2-git-send-email-jmd@gnu.org> (John Darrington's message of "Wed, 22 Feb 2017 13:04:31 +0100")
Hello John,
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?
> diff --git a/bootstrap b/bootstrap
> index cb774bc73..052c433a0 100755
> --- a/bootstrap
> +++ b/bootstrap
> @@ -2,4 +2,13 @@
> # Create the build system.
>
> set -e -x
> -exec autoreconf -vfi
> +ref=$(grep '\\def\\texinfoversion' build-aux/texinfo.tex,ref | tr -c -d [:digit:])
> +autoreconf -vfi
> +ac=$(grep '\\def\\texinfoversion' build-aux/texinfo.tex | tr -c -d [:digit:])
> +
> +# Use our reference version of texinfo.tex or the one from automake, whichever
> +# is most recent.
> +if [ $ref -gt $ac ] ; then
> + cp -f build-aux/texinfo.tex,ref build-aux/texinfo.tex;
> +fi
What about checking in texinfo.tex and removing the -f (--force) option
when invoking 'autoreconf'? This would ensure that texinfo.tex is not
overwritten.
WDYT?
Thanks.
--
Mathieu Lirzin
GPG: F2A3 8D7E EB2B 6640 5761 070D 0ADE E100 9460 4D37
next prev parent reply other threads:[~2017-02-24 19:25 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-22 12:04 Fix the inability to build the manual John Darrington
2017-02-22 12:04 ` [PATCH 1/2] build: Use an up to date copy of texinfo.tex John Darrington
2017-02-24 18:26 ` Leo Famulari
2017-02-24 18:52 ` John Darrington
2017-02-24 19:25 ` Mathieu Lirzin [this message]
2017-02-24 20:12 ` John Darrington
2017-02-24 20:54 ` Mathieu Lirzin
2017-02-25 2:00 ` John Darrington
2017-03-11 11:26 ` Ludovic Courtès
2017-03-29 6:23 ` Chris Marusich
2017-02-22 12:04 ` [PATCH 2/2] doc: Use an approximation to the IPA except for plain output John Darrington
2017-02-24 18:24 ` Leo Famulari
2017-02-24 18:26 ` Fix the inability to build the manual Leo Famulari
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87r32ntmqn.fsf@gnu.org \
--to=mthl@gnu.org \
--cc=guix-devel@gnu.org \
--cc=jmd@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/guix.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.