* problems re: makeinfo '--split-size' arg?
@ 2003-04-28 17:50 cgd
2003-04-28 18:55 ` Joseph S. Myers
2003-04-29 14:06 ` problems re: makeinfo '--split-size' arg? Gerald Pfeifer
0 siblings, 2 replies; 17+ messages in thread
From: cgd @ 2003-04-28 17:50 UTC (permalink / raw)
To: binutils, gcc, pfeifer
So, i just tried building the head of the recently-created binutils
release branch (for target mips-elf, host x86-linux), and ran into:
/path/.../makeinfo: unrecognized option `--split-size=5000000'
My makeinfo has version:
> makeinfo (GNU texinfo) 4.0
>
> Copyright (C) 1999 Free Software Foundation, Inc.
> There is NO warranty. You may redistribute this software
> under the terms of the GNU General Public License.
> For more information about these matters, see the files named COPYING.
And the toplevel 'Makefile.tpl' says:
# For an installed makeinfo, we require it to be from texinfo 4 or
# higher, else we use the "missing" dummy.
While I realize that i should probably be using a newer 'makeinfo', I
cannot easily do so at this time.
I suspect that the top-level Makefile should be modified so that
it uses 'missing' instead of makeinfo if the --split-size option isn't
supported. (Looks like it was added in rev 4.1 of the texinfo
package.)
I suspect that a diff like the one below is in order. I tried it, and
it seems to DTRT for the version of makeinfo that i have (which is all
i tried 8-).
If approved, could somebody else please check it in for me? (I hacked
it into my Makefile.in by hand to test it, but i don't have autogen
and wouldn't want to check in an improperly-generated file, etc...)
thanks,
chris
--
2003-04-28 Chris Demetriou <cgd@broadcom.com>
* Makefile.tpl (MAKEINFO): Require makeinfo 4.1 and later.
* Makefile.in: Regenerate.
Index: Makefile.tpl
===================================================================
RCS file: /cvs/src/src/Makefile.tpl,v
retrieving revision 1.56.4.1
diff -u -p -r1.56.4.1 Makefile.tpl
--- Makefile.tpl 28 Apr 2003 02:25:36 -0000 1.56.4.1
+++ Makefile.tpl 28 Apr 2003 17:49:39 -0000
@@ -194,13 +194,13 @@ M4 = `if [ -f $$r/m4/m4 ] ; \
then echo $$r/m4/m4 ; \
else echo ${DEFAULT_M4} ; fi`
-# For an installed makeinfo, we require it to be from texinfo 4 or
+# For an installed makeinfo, we require it to be from texinfo 4.1 or
# higher, else we use the "missing" dummy.
MAKEINFO=@MAKEINFO@
USUAL_MAKEINFO = `if [ -f $$r/texinfo/makeinfo/makeinfo ] ; \
then echo $$r/texinfo/makeinfo/makeinfo ; \
else if (makeinfo --version \
- | egrep 'texinfo[^0-9]*([1-3][0-9]|[4-9])') >/dev/null 2>&1; \
+ | egrep 'texinfo[^0-9]*([1-3][0-9]|4\.[1-9]|[5-9])') >/dev/null 2>&1; \
then echo makeinfo; else echo $$s/missing makeinfo; fi; fi`
# This just becomes part of the MAKEINFO definition passed down to
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: problems re: makeinfo '--split-size' arg? 2003-04-28 17:50 problems re: makeinfo '--split-size' arg? cgd @ 2003-04-28 18:55 ` Joseph S. Myers [not found] ` <mailpost.1051556216.2781@news-sj1-1> 2003-04-29 14:06 ` problems re: makeinfo '--split-size' arg? Gerald Pfeifer 1 sibling, 1 reply; 17+ messages in thread From: Joseph S. Myers @ 2003-04-28 18:55 UTC (permalink / raw) To: cgd; +Cc: binutils, gcc, pfeifer On Mon, 28 Apr 2003 cgd@broadcom.com wrote: > While I realize that i should probably be using a newer 'makeinfo', I > cannot easily do so at this time. And the binutils manuals "should" be using the more recently added Texinfo @copying feature (see the NEWS file in Texinfo 4.2 or later which says you should switch to it "as soon as possible (as in your next release)" to get proper copyright notices in HTML versions of the manuals). > I suspect that the top-level Makefile should be modified so that > it uses 'missing' instead of makeinfo if the --split-size option isn't > supported. (Looks like it was added in rev 4.1 of the texinfo > package.) For GCC it would be convenient to move the configure check for the required Texinfo version (currently >= 4.2) to toplevel so features from the same version of Texinfo can freely be used in all manuals, not just those in the gcc/ subdirectory. But other projects might not want the required version to change whenever it's convenient to use some new feature in some GCC manual and so the version needed for GCC is increased. -- Joseph S. Myers jsm28@cam.ac.uk ^ permalink raw reply [flat|nested] 17+ messages in thread
[parent not found: <mailpost.1051556216.2781@news-sj1-1>]
* Re: problems re: makeinfo '--split-size' arg? [not found] ` <mailpost.1051556216.2781@news-sj1-1> @ 2003-05-01 17:27 ` cgd 2003-05-02 6:23 ` Gerald Pfeifer 0 siblings, 1 reply; 17+ messages in thread From: cgd @ 2003-05-01 17:27 UTC (permalink / raw) To: jsm28; +Cc: binutils, gcc, pfeifer At Mon, 28 Apr 2003 18:56:56 +0000 (UTC), "Joseph S. Myers" wrote: > > I suspect that the top-level Makefile should be modified so that > > it uses 'missing' instead of makeinfo if the --split-size option isn't > > supported. (Looks like it was added in rev 4.1 of the texinfo > > package.) > > For GCC it would be convenient to move the configure check for the > required Texinfo version (currently >= 4.2) to toplevel so features from > the same version of Texinfo can freely be used in all manuals, not just > those in the gcc/ subdirectory. But other projects might not want the > required version to change whenever it's convenient to use some new > feature in some GCC manual and so the version needed for GCC is increased. It doesn't seem nice to mandate a frequently-increasing version being required across the whole source tree, unless that version is *really* necessary across the source tree programs. On the other hand, there's something to be said for rebuilding info files consistently across the source tree (i.e., if they're not rebuilt in one place, they're not rebuilt in all). If people think that a patch like mine, but w/ the version bumped to 4.2, is reasonable, then that's fine w/ me... All i know is that the current state allows one to use makeinfo 4.0, which fails during the build. 8-) cgd ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: problems re: makeinfo '--split-size' arg? 2003-05-01 17:27 ` cgd @ 2003-05-02 6:23 ` Gerald Pfeifer 2003-05-02 21:44 ` [patch] cgd 2003-05-02 21:45 ` [patch] require makeinfo 4.2 or better cgd 0 siblings, 2 replies; 17+ messages in thread From: Gerald Pfeifer @ 2003-05-02 6:23 UTC (permalink / raw) To: cgd; +Cc: Joseph S. Myers, binutils, gcc On Thu, 1 May 2003 cgd@broadcom.com wrote: > If people think that a patch like mine, but w/ the version bumped to > 4.2, is reasonable, then that's fine w/ me... All i know is that the > current state allows one to use makeinfo 4.0, which fails during the > build. 8-) That's certainly a bug, agreed. Would you mind submitting an updated version of your patch so that the GCC Makefile/configury maintainers can have a look? Thanks, Gerald -- Gerald "Jerry" pfeifer@dbai.tuwien.ac.at http://www.pfeifer.com/gerald/ ^ permalink raw reply [flat|nested] 17+ messages in thread
* [patch] 2003-05-02 6:23 ` Gerald Pfeifer @ 2003-05-02 21:44 ` cgd 2003-05-02 21:45 ` [patch] require makeinfo 4.2 or better cgd 1 sibling, 0 replies; 17+ messages in thread From: cgd @ 2003-05-02 21:44 UTC (permalink / raw) To: Gerald Pfeifer; +Cc: Joseph S. Myers, binutils, gcc At Fri, 2 May 2003 08:23:00 +0200 (CEST), Gerald Pfeifer wrote: > Would you mind submitting an updated > version of your patch so that the GCC Makefile/configury maintainers > can have a look? It's below. And this time i've sent it to the right place. (my original msg wasn't going to include a patch, and i forgot to change gcc -> gcc-patches when i added the patch. 8-) I don't have autogen, so i just applied the same patch to makefile.in, and tested the result w/ texinfo 4.0 to build mipsisa64-elf. I've not tested this w/ a 'good' version of texinfo / makeinfo. If approved, I'd appreciate it if somebody else would apply this (since i'd rather have them actually regenerate Makefile.in than have me hack it. 8-) cgd -- 2003-05-02 Chris Demetriou <cgd@broadcom.com> * Makefile.tpl: Require "makeinfo" from texinfo 4.2 or later. * Makefile.in: Regenerate. Index: Makefile.tpl =================================================================== RCS file: /cvs/gcc/gcc/Makefile.tpl,v retrieving revision 1.44 diff -u -p -r1.44 Makefile.tpl --- Makefile.tpl 28 Apr 2003 02:23:46 -0000 1.44 +++ Makefile.tpl 2 May 2003 21:10:25 -0000 @@ -194,13 +194,13 @@ M4 = `if [ -f $$r/m4/m4 ] ; \ then echo $$r/m4/m4 ; \ else echo ${DEFAULT_M4} ; fi` -# For an installed makeinfo, we require it to be from texinfo 4 or +# For an installed makeinfo, we require it to be from texinfo 4.2 or # higher, else we use the "missing" dummy. MAKEINFO=@MAKEINFO@ USUAL_MAKEINFO = `if [ -f $$r/texinfo/makeinfo/makeinfo ] ; \ then echo $$r/texinfo/makeinfo/makeinfo ; \ else if (makeinfo --version \ - | egrep 'texinfo[^0-9]*([1-3][0-9]|[4-9])') >/dev/null 2>&1; \ + | egrep 'texinfo[^0-9]*([1-3][0-9]|4\.[2-9]|[5-9])') >/dev/null 2>&1; \ then echo makeinfo; else echo $$s/missing makeinfo; fi; fi` # This just becomes part of the MAKEINFO definition passed down to ^ permalink raw reply [flat|nested] 17+ messages in thread
* [patch] require makeinfo 4.2 or better 2003-05-02 6:23 ` Gerald Pfeifer 2003-05-02 21:44 ` [patch] cgd @ 2003-05-02 21:45 ` cgd 1 sibling, 0 replies; 17+ messages in thread From: cgd @ 2003-05-02 21:45 UTC (permalink / raw) To: Gerald Pfeifer; +Cc: Joseph S. Myers, binutils, gcc At Fri, 2 May 2003 08:23:00 +0200 (CEST), Gerald Pfeifer wrote: > Would you mind submitting an updated > version of your patch so that the GCC Makefile/configury maintainers > can have a look? It's below. And this time i've sent it to the right place. (my original msg wasn't going to include a patch, and i forgot to change gcc -> gcc-patches when i added the patch. 8-) I don't have autogen, so i just applied the same patch to makefile.in, and tested the result w/ texinfo 4.0 to build mipsisa64-elf. I've not tested this w/ a 'good' version of texinfo / makeinfo. If approved, I'd appreciate it if somebody else would apply this (since i'd rather have them actually regenerate Makefile.in than have me hack it. 8-) cgd -- 2003-05-02 Chris Demetriou <cgd@broadcom.com> * Makefile.tpl: Require "makeinfo" from texinfo 4.2 or later. * Makefile.in: Regenerate. Index: Makefile.tpl =================================================================== RCS file: /cvs/gcc/gcc/Makefile.tpl,v retrieving revision 1.44 diff -u -p -r1.44 Makefile.tpl --- Makefile.tpl 28 Apr 2003 02:23:46 -0000 1.44 +++ Makefile.tpl 2 May 2003 21:10:25 -0000 @@ -194,13 +194,13 @@ M4 = `if [ -f $$r/m4/m4 ] ; \ then echo $$r/m4/m4 ; \ else echo ${DEFAULT_M4} ; fi` -# For an installed makeinfo, we require it to be from texinfo 4 or +# For an installed makeinfo, we require it to be from texinfo 4.2 or # higher, else we use the "missing" dummy. MAKEINFO=@MAKEINFO@ USUAL_MAKEINFO = `if [ -f $$r/texinfo/makeinfo/makeinfo ] ; \ then echo $$r/texinfo/makeinfo/makeinfo ; \ else if (makeinfo --version \ - | egrep 'texinfo[^0-9]*([1-3][0-9]|[4-9])') >/dev/null 2>&1; \ + | egrep 'texinfo[^0-9]*([1-3][0-9]|4\.[2-9]|[5-9])') >/dev/null 2>&1; \ then echo makeinfo; else echo $$s/missing makeinfo; fi; fi` # This just becomes part of the MAKEINFO definition passed down to ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: problems re: makeinfo '--split-size' arg? 2003-04-28 17:50 problems re: makeinfo '--split-size' arg? cgd 2003-04-28 18:55 ` Joseph S. Myers @ 2003-04-29 14:06 ` Gerald Pfeifer 2003-04-29 14:13 ` Steven Bosscher 1 sibling, 1 reply; 17+ messages in thread From: Gerald Pfeifer @ 2003-04-29 14:06 UTC (permalink / raw) To: cgd; +Cc: binutils, gcc, gcc-patches On Mon, 28 Apr 2003 cgd@broadcom.com wrote: >> makeinfo (GNU texinfo) 4.0 As Joseph (who is the expert on that) wrote we actually require makeinfo 4.2 for GCC and I agree that it should we checked automatically, as per your patch. Still, what I can do, and have in fact considered for the 3.3 branch also before, is using --no-split instead of --split-size, as it shouldn't make a difference there (except, in fact working around a "feature" of --split-size in currently released versions of texinfo). Bootstrapped and installed the 3.3-branch on i386-unknown-freebsd4.8, committed to the 3.3-branch. Gerald 2003-04-29 Gerald Pfeifer <pfeifer@dbai.tuwien.ac.at> * Makefile.tpl (MAKEINFOFLAGS): Use --no-split instead of --split-size. * Makefile.in: Regenerate. Index: Makefile.in =================================================================== RCS file: /cvs/gcc/gcc/Makefile.in,v retrieving revision 1.129.2.5 diff -u -3 -p -r1.129.2.5 Makefile.in --- Makefile.in 25 Apr 2003 20:27:39 -0000 1.129.2.5 +++ Makefile.in 29 Apr 2003 14:04:18 -0000 @@ -163,7 +163,7 @@ MAKEINFO = `if [ -f $$r/texinfo/makeinfo # sub-makes. It lets flags be given on the command line while still # using the makeinfo from the object tree. # (Default to avoid splitting info files by setting the threshold high.) -MAKEINFOFLAGS = --split-size=5000000 +MAKEINFOFLAGS = --no-split EXPECT = `if [ -f $$r/expect/expect ] ; \ then echo $$r/expect/expect ; \ Index: Makefile.tpl =================================================================== RCS file: /cvs/gcc/gcc/Makefile.tpl,v retrieving revision 1.13.6.4 diff -u -3 -p -r1.13.6.4 Makefile.tpl --- Makefile.tpl 25 Apr 2003 20:27:39 -0000 1.13.6.4 +++ Makefile.tpl 29 Apr 2003 14:04:19 -0000 @@ -166,7 +166,7 @@ MAKEINFO = `if [ -f $$r/texinfo/makeinfo # sub-makes. It lets flags be given on the command line while still # using the makeinfo from the object tree. # (Default to avoid splitting info files by setting the threshold high.) -MAKEINFOFLAGS = --split-size=5000000 +MAKEINFOFLAGS = --no-split EXPECT = `if [ -f $$r/expect/expect ] ; \ then echo $$r/expect/expect ; \ ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: problems re: makeinfo '--split-size' arg? 2003-04-29 14:06 ` problems re: makeinfo '--split-size' arg? Gerald Pfeifer @ 2003-04-29 14:13 ` Steven Bosscher 2003-04-29 15:04 ` Gerald Pfeifer 0 siblings, 1 reply; 17+ messages in thread From: Steven Bosscher @ 2003-04-29 14:13 UTC (permalink / raw) To: Gerald Pfeifer; +Cc: cgd, binutils, gcc-patches Op di 29-04-2003, om 16:06 schreef Gerald Pfeifer: > Index: Makefile.in > =================================================================== > RCS file: /cvs/gcc/gcc/Makefile.in,v > retrieving revision 1.129.2.5 > diff -u -3 -p -r1.129.2.5 Makefile.in > --- Makefile.in 25 Apr 2003 20:27:39 -0000 1.129.2.5 > +++ Makefile.in 29 Apr 2003 14:04:18 -0000 > @@ -163,7 +163,7 @@ MAKEINFO = `if [ -f $$r/texinfo/makeinfo > # sub-makes. It lets flags be given on the command line while still > # using the makeinfo from the object tree. > # (Default to avoid splitting info files by setting the threshold high.) > -MAKEINFOFLAGS = --split-size=5000000 > +MAKEINFOFLAGS = --no-split I suppose the comment should be changed here, then? We don't set a threshold, so the comment doesn't make sense anymore. > > EXPECT = `if [ -f $$r/expect/expect ] ; \ > then echo $$r/expect/expect ; \ > Index: Makefile.tpl > =================================================================== > RCS file: /cvs/gcc/gcc/Makefile.tpl,v > retrieving revision 1.13.6.4 > diff -u -3 -p -r1.13.6.4 Makefile.tpl > --- Makefile.tpl 25 Apr 2003 20:27:39 -0000 1.13.6.4 > +++ Makefile.tpl 29 Apr 2003 14:04:19 -0000 > @@ -166,7 +166,7 @@ MAKEINFO = `if [ -f $$r/texinfo/makeinfo > # sub-makes. It lets flags be given on the command line while still > # using the makeinfo from the object tree. > # (Default to avoid splitting info files by setting the threshold high.) > -MAKEINFOFLAGS = --split-size=5000000 > +MAKEINFOFLAGS = --no-split Same here. > > EXPECT = `if [ -f $$r/expect/expect ] ; \ > then echo $$r/expect/expect ; \ > Greetz Steven ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: problems re: makeinfo '--split-size' arg? 2003-04-29 14:13 ` Steven Bosscher @ 2003-04-29 15:04 ` Gerald Pfeifer 0 siblings, 0 replies; 17+ messages in thread From: Gerald Pfeifer @ 2003-04-29 15:04 UTC (permalink / raw) To: Steven Bosscher; +Cc: cgd, binutils, gcc-patches On Tue, 29 Apr 2003, Steven Bosscher wrote: > I suppose the comment should be changed here, then? We don't set a > threshold, so the comment doesn't make sense anymore. Fixed. Gerald -- Gerald "Jerry" pfeifer@dbai.tuwien.ac.at http://www.pfeifer.com/gerald/ ^ permalink raw reply [flat|nested] 17+ messages in thread
[parent not found: <yov5znm4df6m.fsf@broadcom.com>]
[parent not found: <200305030112.h431CbA00420@greed.delorie.com>]
[parent not found: <mailpost.1051924375.21845@news-sj1-1>]
* Re: [patch] require makeinfo 4.2 or better [not found] ` <mailpost.1051924375.21845@news-sj1-1> @ 2003-05-03 1:33 ` cgd 2003-05-03 1:53 ` Daniel Jacobowitz 2003-05-03 6:58 ` Mark Mitchell 0 siblings, 2 replies; 17+ messages in thread From: cgd @ 2003-05-03 1:33 UTC (permalink / raw) To: mark, drow; +Cc: gcc-patches, dj, binutils At Sat, 3 May 2003 01:12:55 +0000 (UTC), "DJ Delorie" wrote: > I think for this patch, manually applying it to both Makefile.tpl and > Makefile.in would be acceptable. > > Oh, and... approved ;-) Thanks, i've applied to both gcc and src repositories. I suppose with the recent changes to use --no-split instead, this patch may not be strictly necessary on the GCC 3.3 branch, but it could be a boon to people building combined-tree builds with gcc + binutils 2.14 (and whatever texinfo happens to be installed on their systems). An alternative is to convert the binutils release branch Makefile.{in,tpl} to use --no-split instead (or to use both patches 8-), i guess. Thoughts? Mark? How about this for the gcc 3.3 branch? (And, now, or after 3.3? 8-) Daniel, which (or both) would you like to see on the binutils branch? cgd -- 2003-05-02 Chris Demetriou <cgd@broadcom.com> * Makefile.tpl: Require "makeinfo" from texinfo 4.2 or later. * Makefile.in: Regenerate. Index: Makefile.tpl =================================================================== RCS file: /cvs/gcc/gcc/Makefile.tpl,v retrieving revision 1.44 diff -u -p -r1.44 Makefile.tpl --- Makefile.tpl 28 Apr 2003 02:23:46 -0000 1.44 +++ Makefile.tpl 2 May 2003 21:10:25 -0000 @@ -194,13 +194,13 @@ M4 = `if [ -f $$r/m4/m4 ] ; \ then echo $$r/m4/m4 ; \ else echo ${DEFAULT_M4} ; fi` -# For an installed makeinfo, we require it to be from texinfo 4 or +# For an installed makeinfo, we require it to be from texinfo 4.2 or # higher, else we use the "missing" dummy. MAKEINFO=@MAKEINFO@ USUAL_MAKEINFO = `if [ -f $$r/texinfo/makeinfo/makeinfo ] ; \ then echo $$r/texinfo/makeinfo/makeinfo ; \ else if (makeinfo --version \ - | egrep 'texinfo[^0-9]*([1-3][0-9]|[4-9])') >/dev/null 2>&1; \ + | egrep 'texinfo[^0-9]*([1-3][0-9]|4\.[2-9]|[5-9])') >/dev/null 2>&1; \ then echo makeinfo; else echo $$s/missing makeinfo; fi; fi` # This just becomes part of the MAKEINFO definition passed down to ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [patch] require makeinfo 4.2 or better 2003-05-03 1:33 ` [patch] require makeinfo 4.2 or better cgd @ 2003-05-03 1:53 ` Daniel Jacobowitz 2003-05-03 2:42 ` cgd 2003-05-03 6:58 ` Mark Mitchell 1 sibling, 1 reply; 17+ messages in thread From: Daniel Jacobowitz @ 2003-05-03 1:53 UTC (permalink / raw) To: cgd; +Cc: mark, gcc-patches, dj, binutils On Fri, May 02, 2003 at 06:33:39PM -0700, cgd@broadcom.com wrote: > At Sat, 3 May 2003 01:12:55 +0000 (UTC), "DJ Delorie" wrote: > > I think for this patch, manually applying it to both Makefile.tpl and > > Makefile.in would be acceptable. > > > > Oh, and... approved ;-) > > Thanks, i've applied to both gcc and src repositories. > > I suppose with the recent changes to use --no-split instead, this > patch may not be strictly necessary on the GCC 3.3 branch, but it > could be a boon to people building combined-tree builds with gcc + > binutils 2.14 (and whatever texinfo happens to be installed on their > systems). > > An alternative is to convert the binutils release branch > Makefile.{in,tpl} to use --no-split instead (or to use both patches > 8-), i guess. > > Thoughts? > > Mark? How about this for the gcc 3.3 branch? (And, now, or after > 3.3? 8-) > > Daniel, which (or both) would you like to see on the binutils branch? I haven't been paying attention. Why does binutils need it? -- Daniel Jacobowitz MontaVista Software Debian GNU/Linux Developer ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [patch] require makeinfo 4.2 or better 2003-05-03 1:53 ` Daniel Jacobowitz @ 2003-05-03 2:42 ` cgd 2003-05-03 4:55 ` Daniel Jacobowitz 0 siblings, 1 reply; 17+ messages in thread From: cgd @ 2003-05-03 2:42 UTC (permalink / raw) To: Daniel Jacobowitz; +Cc: mark, gcc-patches, dj, binutils At Fri, 2 May 2003 21:53:34 -0400, Daniel Jacobowitz wrote: > > Daniel, which (or both) would you like to see on the binutils branch? > > I haven't been paying attention. Why does binutils need it? heh. using with installed makeinfo 4.0 in path: makeinfo --split-size=5000000 -I../../src/libiberty ../../src/libiberty/libiberty.texi makeinfo: unrecognized option `--split-size=5000000' Try `makeinfo --help' for more information. [...] gmake[1]: *** [libiberty.info] Error 1 8-) options are: (1) remove --split-size use (go with --no-split instead), which is compatible with earlier texinfo. (This which has been done on the gcc 3.3 branch, but *not* on trunk.) (2) require texinfo 4.2 or later. (done on gcc & src trunk.) (3) both. (1) gets rid of the error above, but apparently there are other reasons (they've been kicked out of my memory, see the thread on gcc@ 8-) which 4.2 and later are desirable. cgd ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [patch] require makeinfo 4.2 or better 2003-05-03 2:42 ` cgd @ 2003-05-03 4:55 ` Daniel Jacobowitz 2003-05-03 6:44 ` cgd 2003-05-03 22:11 ` cgd 0 siblings, 2 replies; 17+ messages in thread From: Daniel Jacobowitz @ 2003-05-03 4:55 UTC (permalink / raw) To: cgd; +Cc: mark, gcc-patches, dj, binutils On Fri, May 02, 2003 at 07:41:50PM -0700, cgd@broadcom.com wrote: > At Fri, 2 May 2003 21:53:34 -0400, Daniel Jacobowitz wrote: > > > Daniel, which (or both) would you like to see on the binutils branch? > > > > I haven't been paying attention. Why does binutils need it? > > heh. using with installed makeinfo 4.0 in path: > > makeinfo --split-size=5000000 -I../../src/libiberty ../../src/libiberty/libiberty.texi > makeinfo: unrecognized option `--split-size=5000000' > Try `makeinfo --help' for more information. > [...] > gmake[1]: *** [libiberty.info] Error 1 > > > 8-) > > options are: > > (1) remove --split-size use (go with --no-split instead), which is > compatible with earlier texinfo. (This which has been done on the > gcc 3.3 branch, but *not* on trunk.) > > (2) require texinfo 4.2 or later. (done on gcc & src trunk.) > > (3) both. > > > (1) gets rid of the error above, but apparently there are other > reasons (they've been kicked out of my memory, see the thread on gcc@ > 8-) which 4.2 and later are desirable. Hmm. For now, let's go for --no-split. Would you mind fixing the branch? -- Daniel Jacobowitz MontaVista Software Debian GNU/Linux Developer ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [patch] require makeinfo 4.2 or better 2003-05-03 4:55 ` Daniel Jacobowitz @ 2003-05-03 6:44 ` cgd 2003-05-03 22:11 ` cgd 1 sibling, 0 replies; 17+ messages in thread From: cgd @ 2003-05-03 6:44 UTC (permalink / raw) To: Daniel Jacobowitz; +Cc: mark, gcc-patches, dj, binutils At Sat, 3 May 2003 00:55:19 -0400, Daniel Jacobowitz wrote: > [ binutils release branch makeinfo ] > Hmm. For now, let's go for --no-split. Would you mind fixing the > branch? Sure, i'll try to get to it tmrw or sunday. frying other fish tonight. 8-) chris ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [patch] require makeinfo 4.2 or better 2003-05-03 4:55 ` Daniel Jacobowitz 2003-05-03 6:44 ` cgd @ 2003-05-03 22:11 ` cgd 1 sibling, 0 replies; 17+ messages in thread From: cgd @ 2003-05-03 22:11 UTC (permalink / raw) To: Daniel Jacobowitz; +Cc: mark, gcc-patches, dj, binutils At Sat, 3 May 2003 00:55:19 -0400, Daniel Jacobowitz wrote: > [ binutils branch solution ] > Hmm. For now, let's go for --no-split. Would you mind fixing the > branch? I've done so. cgd ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [patch] require makeinfo 4.2 or better 2003-05-03 1:33 ` [patch] require makeinfo 4.2 or better cgd 2003-05-03 1:53 ` Daniel Jacobowitz @ 2003-05-03 6:58 ` Mark Mitchell 2003-05-03 18:11 ` cgd 1 sibling, 1 reply; 17+ messages in thread From: Mark Mitchell @ 2003-05-03 6:58 UTC (permalink / raw) To: cgd; +Cc: drow, gcc-patches, dj, binutils On Fri, 2003-05-02 at 18:33, cgd@broadcom.com wrote: > At Sat, 3 May 2003 01:12:55 +0000 (UTC), "DJ Delorie" wrote: > > I think for this patch, manually applying it to both Makefile.tpl and > > Makefile.in would be acceptable. > > > > Oh, and... approved ;-) > > Thanks, i've applied to both gcc and src repositories. > > I suppose with the recent changes to use --no-split instead, this > patch may not be strictly necessary on the GCC 3.3 branch, but it > could be a boon to people building combined-tree builds with gcc + > binutils 2.14 (and whatever texinfo happens to be installed on their > systems). > > An alternative is to convert the binutils release branch > Makefile.{in,tpl} to use --no-split instead (or to use both patches > 8-), i guess. > > Thoughts? > > Mark? How about this for the gcc 3.3 branch? (And, now, or after > 3.3? 8-) This is OK for 3.3; check it in ASAP. Thanks, -- Mark Mitchell <mark@codesourcery.com> CodeSourcery, LLC ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [patch] require makeinfo 4.2 or better 2003-05-03 6:58 ` Mark Mitchell @ 2003-05-03 18:11 ` cgd 0 siblings, 0 replies; 17+ messages in thread From: cgd @ 2003-05-03 18:11 UTC (permalink / raw) To: Mark Mitchell; +Cc: drow, gcc-patches, dj, binutils At 02 May 2003 23:58:26 -0700, Mark Mitchell wrote: > > Mark? How about this for the gcc 3.3 branch? (And, now, or after > > 3.3? 8-) > > This is OK for 3.3; check it in ASAP. OK. The exact patch didn't apply (looks like USUAL_MAKEINFO, etc., was added on the trunk), but the merge was trivial. 8-) Below is exactly what was applied. cgd -- 2003-05-03 Chris Demetriou <cgd@broadcom.com> Merge from mainline: 2003-05-02 Chris Demetriou <cgd@broadcom.com> * Makefile.tpl: Require "makeinfo" from texinfo 4.2 or later. * Makefile.in: Regenerate. Index: Makefile.in =================================================================== RCS file: /cvs/gcc/gcc/Makefile.in,v retrieving revision 1.129.2.7 diff -u -p -r1.129.2.7 Makefile.in --- Makefile.in 29 Apr 2003 15:03:59 -0000 1.129.2.7 +++ Makefile.in 3 May 2003 18:07:48 -0000 @@ -151,12 +151,12 @@ M4 = `if [ -f $$r/m4/m4 ] ; \ then echo $$r/m4/m4 ; \ else echo ${DEFAULT_M4} ; fi` -# For an installed makeinfo, we require it to be from texinfo 4 or +# For an installed makeinfo, we require it to be from texinfo 4.2 or # higher, else we use the "missing" dummy. MAKEINFO = `if [ -f $$r/texinfo/makeinfo/makeinfo ] ; \ then echo $$r/texinfo/makeinfo/makeinfo ; \ else if (makeinfo --version \ - | egrep 'texinfo[^0-9]*([1-3][0-9]|[4-9])') >/dev/null 2>&1; \ + | egrep 'texinfo[^0-9]*([1-3][0-9]|4\.[2-9]|[5-9])') >/dev/null 2>&1; \ then echo makeinfo; else echo $$s/missing makeinfo; fi; fi` # This just becomes part of the MAKEINFO definition passed down to Index: Makefile.tpl =================================================================== RCS file: /cvs/gcc/gcc/Makefile.tpl,v retrieving revision 1.13.6.6 diff -u -p -r1.13.6.6 Makefile.tpl --- Makefile.tpl 29 Apr 2003 15:03:59 -0000 1.13.6.6 +++ Makefile.tpl 3 May 2003 18:07:48 -0000 @@ -154,12 +154,12 @@ M4 = `if [ -f $$r/m4/m4 ] ; \ then echo $$r/m4/m4 ; \ else echo ${DEFAULT_M4} ; fi` -# For an installed makeinfo, we require it to be from texinfo 4 or +# For an installed makeinfo, we require it to be from texinfo 4.2 or # higher, else we use the "missing" dummy. MAKEINFO = `if [ -f $$r/texinfo/makeinfo/makeinfo ] ; \ then echo $$r/texinfo/makeinfo/makeinfo ; \ else if (makeinfo --version \ - | egrep 'texinfo[^0-9]*([1-3][0-9]|[4-9])') >/dev/null 2>&1; \ + | egrep 'texinfo[^0-9]*([1-3][0-9]|4\.[2-9]|[5-9])') >/dev/null 2>&1; \ then echo makeinfo; else echo $$s/missing makeinfo; fi; fi` # This just becomes part of the MAKEINFO definition passed down to ^ permalink raw reply [flat|nested] 17+ messages in thread
end of thread, other threads:[~2003-05-03 22:11 UTC | newest] Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2003-04-28 17:50 problems re: makeinfo '--split-size' arg? cgd 2003-04-28 18:55 ` Joseph S. Myers [not found] ` <mailpost.1051556216.2781@news-sj1-1> 2003-05-01 17:27 ` cgd 2003-05-02 6:23 ` Gerald Pfeifer 2003-05-02 21:44 ` [patch] cgd 2003-05-02 21:45 ` [patch] require makeinfo 4.2 or better cgd 2003-04-29 14:06 ` problems re: makeinfo '--split-size' arg? Gerald Pfeifer 2003-04-29 14:13 ` Steven Bosscher 2003-04-29 15:04 ` Gerald Pfeifer [not found] <yov5znm4df6m.fsf@broadcom.com> [not found] ` <200305030112.h431CbA00420@greed.delorie.com> [not found] ` <mailpost.1051924375.21845@news-sj1-1> 2003-05-03 1:33 ` [patch] require makeinfo 4.2 or better cgd 2003-05-03 1:53 ` Daniel Jacobowitz 2003-05-03 2:42 ` cgd 2003-05-03 4:55 ` Daniel Jacobowitz 2003-05-03 6:44 ` cgd 2003-05-03 22:11 ` cgd 2003-05-03 6:58 ` Mark Mitchell 2003-05-03 18:11 ` cgd
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).