Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* 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