diff mbox

[committed] Fix gdb.texinfo for old makeinfo (again)

Message ID 5A5C95D8.3010405@arm.com
State New
Headers show

Commit Message

Szabolcs Nagy Jan. 15, 2018, 11:51 a.m. UTC
On 15/01/18 11:48, Szabolcs Nagy wrote:
> gdb/doc/ChangeLog:
> 2018-01-15  Szabolcs Nagy  <szabolcs.nagy@arm.com>
> 
> 	* gdb.texinfo (set cwd): Add period.
> 	(gdb-add-index): Move anchor.
> 

now with patch attached.

Comments

Eli Zaretskii Jan. 15, 2018, 12:11 p.m. UTC | #1
> Date: Mon, 15 Jan 2018 11:51:52 +0000
> From: Szabolcs Nagy <szabolcs.nagy@arm.com>
> CC: nd@arm.com, Sergio Durigan Junior <sergiodj@redhat.com>,  Simon Marchi <simon.marchi@polymtl.ca>, Tom Tromey <tom@tromey.com>, Eli Zaretskii <eliz@gnu.org>
> now with patch attached.

Hmm... why did you need this one:

> -@pindex gdb-add-index

That's an index entry for gdb-add-index, it should stay, I think.
Szabolcs Nagy Jan. 15, 2018, 12:25 p.m. UTC | #2
On 15/01/18 12:11, Eli Zaretskii wrote:
>> Date: Mon, 15 Jan 2018 11:51:52 +0000
>> From: Szabolcs Nagy <szabolcs.nagy@arm.com>
>> CC: nd@arm.com, Sergio Durigan Junior <sergiodj@redhat.com>,  Simon Marchi <simon.marchi@polymtl.ca>, Tom Tromey <tom@tromey.com>, Eli Zaretskii <eliz@gnu.org>
>> now with patch attached.
> 
> Hmm... why did you need this one:
> 
>> -@pindex gdb-add-index
> 
> That's an index entry for gdb-add-index, it should stay, I think.
> 

i still get the error if i keep @pindex there,
if i move it together with @anchor below @heading
that works, shall i make that change?

(i just wanted to unbreak the build for our
automated toolchain tests, but there might be
better fix)
Eli Zaretskii Jan. 15, 2018, 12:58 p.m. UTC | #3
> Date: Mon, 15 Jan 2018 12:25:26 +0000
> From: Szabolcs Nagy <szabolcs.nagy@arm.com>
> CC: nd@arm.com, gdb-patches@sourceware.org, sergiodj@redhat.com, 
>  simon.marchi@polymtl.ca, tom@tromey.com
> >> -@pindex gdb-add-index
> > 
> > That's an index entry for gdb-add-index, it should stay, I think.
> > 
> 
> i still get the error if i keep @pindex there,
> if i move it together with @anchor below @heading
> that works, shall i make that change?

Yes, @pindex should be below @heading.

Thanks.

> (i just wanted to unbreak the build for our
> automated toolchain tests

Understood.
diff mbox

Patch

diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo
index b6e7a4975e..75789a27c2 100644
--- a/gdb/doc/gdb.texinfo
+++ b/gdb/doc/gdb.texinfo
@@ -2495,7 +2495,7 @@  fallback.
 
 You can also change @value{GDBN}'s current working directory by using
 the @code{cd} command.
-@xref{cd command}
+@xref{cd command}.
 
 @kindex show cwd
 @cindex show inferior's working directory
@@ -2514,7 +2514,7 @@  given, @var{directory} uses @file{'~'}.
 The @value{GDBN} working directory serves as a default for the
 commands that specify files for @value{GDBN} to operate on.
 @xref{Files, ,Commands to Specify Files}.
-@xref{set cwd command}
+@xref{set cwd command}.
 
 @kindex pwd
 @item pwd
@@ -43441,9 +43441,8 @@  Richard M. Stallman and Roland H. Pesch, July 1991.
 @c man end
 
 @node gdb-add-index man
-@pindex gdb-add-index
-@anchor{gdb-add-index}
 @heading gdb-add-index
+@anchor{gdb-add-index}
 
 @c man title gdb-add-index Add index files to speed up GDB