* doc: Specify the CWD explicitly for gdb-cfg.texi
@ 2007-07-25 16:02 Maciej W. Rozycki
2007-07-25 18:12 ` Daniel Jacobowitz
2007-08-07 16:01 ` Joseph S. Myers
0 siblings, 2 replies; 4+ messages in thread
From: Maciej W. Rozycki @ 2007-07-25 16:02 UTC (permalink / raw)
To: gdb-patches; +Cc: Maciej W. Rozycki
Hello,
Contrary to documented behaviour `texi2html' does not search the current
directory for include files. I get:
*** Can't find gdb-cfg.texi, skipping at /usr/bin/texi2html line 3662, <FH001> line 11.
with version 1.66 and:
*** Can't find gdb-cfg.texi, skipping (l. 11)
with version 1.76. That deserves a bug report against `texi2html', but
the following trivial workaround makes `make html' work as expected with
versions already in existence.
gdb/:
2007-07-25 Maciej W. Rozycki <macro@mips.com>
* Makefile.in (MAKEHTMLFLAGS): Also search the current directory
for include files.
gdb/doc/:
2007-07-25 Maciej W. Rozycki <macro@mips.com>
* Makefile.in (MAKEHTMLFLAGS): Also search the current directory
for include files.
OK to apply?
Maciej
gdb-cfg-texi.diff
Index: gdb/src/gdb/doc/Makefile.in
===================================================================
--- gdb.orig/src/gdb/doc/Makefile.in 2007-07-24 17:41:08.000000000 +0100
+++ gdb/src/gdb/doc/Makefile.in 2007-07-25 15:55:35.000000000 +0100
@@ -49,9 +49,11 @@
# Note that texinfo 4.0's makeinfo --html can only generate a
# single file, which would be too large, so continue to use
# texi2html. -sts 2000-03-28
-
+# Contrary to documentation, the current directory is not
+# searched by texi2html, so specify it explicitly.
+# -macro 2007-07-25
MAKEHTML = texi2html
-MAKEHTMLFLAGS = -menu -split_chapter
+MAKEHTMLFLAGS = -menu -split_chapter -I .
# where to find texi2roff, ditto
TEXI2ROFF=texi2roff
Index: gdb/src/gdb/Makefile.in
===================================================================
--- gdb.orig/src/gdb/Makefile.in 2007-07-24 17:41:09.000000000 +0100
+++ gdb/src/gdb/Makefile.in 2007-07-25 15:55:31.000000000 +0100
@@ -99,9 +99,11 @@
# where to find makeinfo, preferably one designed for texinfo-2
MAKEINFO=makeinfo
+# Contrary to documentation, the current directory is not
+# searched by texi2html, so specify it explicitly.
+# -macro 2007-07-25
MAKEHTML = texi2html
-
-MAKEHTMLFLAGS = -menu -split_chapter
+MAKEHTMLFLAGS = -menu -split_chapter -I .
# Set this up with gcc if you have gnu ld and the loader will print out
# line numbers for undefined references.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: doc: Specify the CWD explicitly for gdb-cfg.texi
2007-07-25 16:02 doc: Specify the CWD explicitly for gdb-cfg.texi Maciej W. Rozycki
@ 2007-07-25 18:12 ` Daniel Jacobowitz
2007-08-07 16:01 ` Joseph S. Myers
1 sibling, 0 replies; 4+ messages in thread
From: Daniel Jacobowitz @ 2007-07-25 18:12 UTC (permalink / raw)
To: Maciej W. Rozycki; +Cc: gdb-patches, Maciej W. Rozycki
On Wed, Jul 25, 2007 at 04:38:01PM +0100, Maciej W. Rozycki wrote:
> Hello,
>
> Contrary to documented behaviour `texi2html' does not search the current
> directory for include files. I get:
>
> *** Can't find gdb-cfg.texi, skipping at /usr/bin/texi2html line 3662, <FH001> line 11.
>
> with version 1.66 and:
>
> *** Can't find gdb-cfg.texi, skipping (l. 11)
>
> with version 1.76. That deserves a bug report against `texi2html', but
> the following trivial workaround makes `make html' work as expected with
> versions already in existence.
OK, thanks.
--
Daniel Jacobowitz
CodeSourcery
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: doc: Specify the CWD explicitly for gdb-cfg.texi
2007-07-25 16:02 doc: Specify the CWD explicitly for gdb-cfg.texi Maciej W. Rozycki
2007-07-25 18:12 ` Daniel Jacobowitz
@ 2007-08-07 16:01 ` Joseph S. Myers
2007-08-09 18:37 ` Carlos O'Donell
1 sibling, 1 reply; 4+ messages in thread
From: Joseph S. Myers @ 2007-08-07 16:01 UTC (permalink / raw)
To: Carlos O'Donell; +Cc: gdb-patches
On Wed, 25 Jul 2007, Maciej W. Rozycki wrote:
> @@ -49,9 +49,11 @@
> # Note that texinfo 4.0's makeinfo --html can only generate a
> # single file, which would be too large, so continue to use
> # texi2html. -sts 2000-03-28
I might note that this comment is very out-of-date by now (HTML output
supports splitting by node since version 4.1, and the src tree's toplevel
requires makeinfo version 4.4 or later). I think it would be a good idea
for GDB to use makeinfo --html just like other GNU software does, and
thereby avoid the various texi2html compatibility issues such as those
addressed by these recent patches. Carlos, perhaps you could contribute
your patch to do so?
--
Joseph S. Myers
joseph@codesourcery.com
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: doc: Specify the CWD explicitly for gdb-cfg.texi
2007-08-07 16:01 ` Joseph S. Myers
@ 2007-08-09 18:37 ` Carlos O'Donell
0 siblings, 0 replies; 4+ messages in thread
From: Carlos O'Donell @ 2007-08-09 18:37 UTC (permalink / raw)
To: Joseph S. Myers; +Cc: gdb-patches
On Tue, Aug 07, 2007 at 04:01:04PM +0000, Joseph S. Myers wrote:
> On Wed, 25 Jul 2007, Maciej W. Rozycki wrote:
>
> > @@ -49,9 +49,11 @@
> > # Note that texinfo 4.0's makeinfo --html can only generate a
> > # single file, which would be too large, so continue to use
> > # texi2html. -sts 2000-03-28
>
> I might note that this comment is very out-of-date by now (HTML output
> supports splitting by node since version 4.1, and the src tree's toplevel
> requires makeinfo version 4.4 or later). I think it would be a good idea
> for GDB to use makeinfo --html just like other GNU software does, and
> thereby avoid the various texi2html compatibility issues such as those
> addressed by these recent patches. Carlos, perhaps you could contribute
> your patch to do so?
I can definately contribute this patch. Let me update it for mainline. I
should post this before the end of the week.
Cheers,
Carlos.
--
Carlos O'Donell
CodeSourcery
carlos@codesourcery.com
(650) 331-3385 x716
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2007-08-09 18:37 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-07-25 16:02 doc: Specify the CWD explicitly for gdb-cfg.texi Maciej W. Rozycki
2007-07-25 18:12 ` Daniel Jacobowitz
2007-08-07 16:01 ` Joseph S. Myers
2007-08-09 18:37 ` Carlos O'Donell
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox