raising minimum Texinfo requirement in src tree

raising minimum Texinfo requirement in src tree

[Ben Elliston]

I've just committed a patch that raises the minimum requirement on
Texinfo from version 4.2 (released in April 2002) to Texinfo 4.4
(released in January 2003).  DJ has advised me to consult with these
projects about the change.  Does anyone have any objections?  If so, I
will back it out and we can discuss.

FWIW, I have been using Texinfo 4.8 on my Debian GNU/Linux system.  I
think this is a pretty conservative change that is needed to support
some fairly modern Texinfo constructs in the binutils documentation.

Thanks, Ben

Re: raising minimum Texinfo requirement in src tree

[Eli Zaretskii]

> Date: Fri, 31 Mar 2006 13:41:35 +1100
> From: Ben Elliston <bje@au1.ibm.com>
> 
> I've just committed a patch that raises the minimum requirement on
> Texinfo from version 4.2 (released in April 2002) to Texinfo 4.4
> (released in January 2003).

Which patch was that?  Can you show it or give a URL?  I don't
remember seeing such a patch on gdb-patches, but my memory is not what
it used to be.

> DJ has advised me to consult with these projects about the change.
> Does anyone have any objections?

Can't make up my mind before I see the patch.

Re: raising minimum Texinfo requirement in src tree

[Ben Elliston]

> Which patch was that?  Can you show it or give a URL?  I don't
> remember seeing such a patch on gdb-patches, but my memory is not
> what it used to be.

2006-03-31  Ben Elliston  <bje@au.ibm.com>

        PR binutils/1860
        * configure.in: Require makeinfo 4.4 or higher.
        * configure: Regenerate.

Index: configure.in
===================================================================
RCS file: /cvs/src/src/configure.in,v
retrieving revision 1.301
retrieving revision 1.302
diff -u -p -r1.301 -r1.302
--- configure.in        14 Mar 2006 16:10:08 -0000      1.301
+++ configure.in        31 Mar 2006 02:32:20 -0000      1.302
@@ -2203,10 +2203,10 @@ case " $build_configdirs " in
   *" texinfo "*) MAKEINFO='$$r/$(BUILD_SUBDIR)/texinfo/makeinfo/makeinfo' ;;
   *)
 changequote(,)
-    # For an installed makeinfo, we require it to be from texinfo 4.2 or
+    # For an installed makeinfo, we require it to be from texinfo 4.4 or
     # higher, else we use the "missing" dummy.
     if ${MAKEINFO} --version \
-       | egrep 'texinfo[^0-9]*([1-3][0-9]|4\.[2-9]|[5-9])' >/dev/null 2>&1; then
+       | egrep 'texinfo[^0-9]*([1-3][0-9]|4\.[4-9]|[5-9])' >/dev/null 2>&1; then
       :
     else
       MAKEINFO="$MISSING makeinfo"

Re: raising minimum Texinfo requirement in src tree

[Eli Zaretskii]

> Date: Mon, 3 Apr 2006 09:54:18 +1000
> From: Ben Elliston <bje@au1.ibm.com>
> Cc: gdb-patches@sourceware.org, newlib@sourceware.org,         cygwin-patches@sourceware.org
> 
> > Which patch was that?  Can you show it or give a URL?  I don't
> > remember seeing such a patch on gdb-patches, but my memory is not
> > what it used to be.
> 
> 2006-03-31  Ben Elliston  <bje@au.ibm.com>
> 
>         PR binutils/1860
>         * configure.in: Require makeinfo 4.4 or higher.
>         * configure: Regenerate.

Thanks, but that doesn't show the _reason_ for requesting Texinfo 4.4.
What was the change in the manual(s) that prompted you to bump the
minimum version?

Re: raising minimum Texinfo requirement in src tree

[Daniel Jacobowitz]

On Mon, Apr 03, 2006 at 06:35:34AM +0300, Eli Zaretskii wrote:
> > 2006-03-31  Ben Elliston  <bje@au.ibm.com>
> > 
> >         PR binutils/1860

> Thanks, but that doesn't show the _reason_ for requesting Texinfo 4.4.
> What was the change in the manual(s) that prompted you to bump the
> minimum version?

Here it is:

  http://sourceware.org/bugzilla/show_bug.cgi?id=1860

I believe it's @include{@value{}}.

-- 
Daniel Jacobowitz
CodeSourcery

Re: raising minimum Texinfo requirement in src tree

[Ben Elliston]

> Here it is:
>   http://sourceware.org/bugzilla/show_bug.cgi?id=1860
> I believe it's @include{@value{}}.

That's right.

Ben

Re: raising minimum Texinfo requirement in src tree

[Mark Kettenis]

> Date: Sun, 2 Apr 2006 23:42:17 -0400
> From: Daniel Jacobowitz <drow@false.org>
> 
> On Mon, Apr 03, 2006 at 06:35:34AM +0300, Eli Zaretskii wrote:
> > > 2006-03-31  Ben Elliston  <bje@au.ibm.com>
> > > 
> > >         PR binutils/1860
> 
> > Thanks, but that doesn't show the _reason_ for requesting Texinfo 4.4.
> > What was the change in the manual(s) that prompted you to bump the
> > minimum version?
> 
> Here it is:
> 
>   http://sourceware.org/bugzilla/show_bug.cgi?id=1860
> 
> I believe it's @include{@value{}}.

Indeed, I hacked the makeinfo in the OpenBSD tree to recognize that
one.  Guess I'll have to do a full import of a more recent makeinfo
now.  Not a big problem though, so no objection to the patch.

Mark