Re: [PATCH 1/2] build: Use an up to date copy of texinfo.tex

[Mathieu Lirzin]

Hello John,

John Darrington <address@hidden> 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