Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: michael.hope@linaro.org
Cc: brobecker@adacore.com, joseph@codesourcery.com,
	gdb-patches@sourceware.org
Subject: Re: [PATCH, doc]: Rename Index node to prevent file collision
Date: Sun, 01 Jul 2012 19:56:00 -0000	[thread overview]
Message-ID: <83r4sv5k9h.fsf@gnu.org> (raw)
In-Reply-To: <837guzd5q1.fsf@gnu.org>

Ping!  Should I commit this?

> Date: Fri, 22 Jun 2012 13:02:46 +0300
> From: Eli Zaretskii <eliz@gnu.org>
> Cc: brobecker@adacore.com, joseph@codesourcery.com, gdb-patches@sourceware.org
> 
> > Date: Fri, 22 Jun 2012 08:58:19 +0300
> > From: Eli Zaretskii <eliz@gnu.org>
> > Cc: brobecker@adacore.com, joseph@codesourcery.com, gdb-patches@sourceware.org
> > 
> > > >> > The problem with your suggestion is that the GDB index is not a
> > > >> > concept index, it is all the indices lumped into one.  But I would be
> > > >> > OK if we separate the concept index from the rest, and then we could
> > > >> > have "Concept Index" and "Command and Variable Index".
> > > >>
> > > >> I'd rather not go there as it's a big change for little gain.
> > > >
> > > > ??? It's as simple as modifying the "@syncodeindex" directives at the
> > > > beginning of gdb.texinfo, and then adding 2 @node lines for the two
> > > > indices, instead of the current one.  All the rest will be done by
> > > > makeinfo.  Am I missing something?
> > > 
> > > Sorry, I assumed that we'd have to check and perhaps update each index
> > > entry to see that it's in the right category.
> > 
> > No.  We already have separate index categories: see the @kindex,
> > @findex, @vindex, etc. directives, in addition to @cindex, that we
> > have all over the manual.  It's just that these 4 directives at the
> > beginning of gdb.texinfo:
> > 
> >   @syncodeindex ky cp
> >   @syncodeindex tp cp
> > 
> >   @c readline appendices use @vindex, @findex and @ftable,
> >   @c annotate.texi and gdbmi use @findex.
> >   @syncodeindex vr cp
> >   @syncodeindex fn cp
> > 
> > arrange for all of them to be lumped into a single index "cp" (which
> > stands for "Concept Index"), and so this directive in the single
> > "Index" node we have:
> > 
> >   @printindex cp
> > 
> > prints all of the index entries together.
> > 
> > > Your texinfo foo is better than mine - could you post a patch?
> > 
> > OK.
> 
> Here it is.  Does this solve your problem?
> 
> 
> Index: gdb.texinfo
> ===================================================================
> RCS file: /cvs/src/src/gdb/doc/gdb.texinfo,v
> retrieving revision 1.978
> diff -u -p -r1.978 gdb.texinfo
> --- gdb.texinfo	22 Jun 2012 08:37:49 -0000	1.978
> +++ gdb.texinfo	22 Jun 2012 10:01:35 -0000
> @@ -18,13 +18,17 @@
>  @end iftex
>  
>  @finalout
> -@syncodeindex ky cp
> -@syncodeindex tp cp
> +@c To avoid file-name clashes between index.html and Index.html, when
> +@c the manual is produced on a Posix host and then moved to a
> +@c case-insensitive filesystem (e.g., MS-Windows), we separate the
> +@c indices into two: Concept Index and all the rest.
> +@syncodeindex ky fn
> +@syncodeindex tp fn
>  
>  @c readline appendices use @vindex, @findex and @ftable,
>  @c annotate.texi and gdbmi use @findex.
> -@syncodeindex vr cp
> -@syncodeindex fn cp
> +@syncodeindex vr fn
> +@syncodeindex fn fn
>  
>  @c !!set GDB manual's edition---not the same as GDB version!
>  @c This is updated by GNU Press.
> @@ -182,7 +186,9 @@ software in general.  We will miss him.
>  * Copying::			GNU General Public License says
>                                  how you can copy and share GDB
>  * GNU Free Documentation License::  The license for this documentation
> -* Index::                       Index
> +* Concept Index::               Index of @value{GDBN} concepts
> +* Command and Variable Index::  Index of @value{GDBN} commands, variables,
> +                                  functions, and Python data types
>  @end menu
>  
>  @end ifnottex
> @@ -1780,8 +1786,9 @@ In addition to @code{help}, you can use 
>  and @code{show} to inquire about the state of your program, or the state
>  of @value{GDBN} itself.  Each command supports many topics of inquiry; this
>  manual introduces each of them in the appropriate context.  The listings
> -under @code{info} and under @code{show} in the Index point to
> -all the sub-commands.  @xref{Index}.
> +under @code{info} and under @code{show} in the Command, Variable, and
> +Function Index point to all the sub-commands.  @xref{Command and Variable
> +Index}.
>  
>  @c @group
>  @table @code
> @@ -40415,11 +40422,16 @@ A string in the constant pool is zero-te
>  @appendix GNU Free Documentation License
>  @include fdl.texi
>  
> -@node Index
> -@unnumbered Index
> +@node Concept Index
> +@unnumbered Concept Index
>  
>  @printindex cp
>  
> +@node Command and Variable Index
> +@unnumbered Command, Variable, and Function Index
> +
> +@printindex fn
> +
>  @tex
>  % I think something like @@colophon should be in texinfo.  In the
>  % meantime:
> 


  reply	other threads:[~2012-07-01 19:56 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-06-14  2:39 Michael Hope
2012-06-14 15:06 ` Joseph S. Myers
2012-06-14 16:42   ` Eli Zaretskii
2012-06-14 21:43     ` Michael Hope
2012-06-14 22:01       ` Joel Brobecker
2012-06-14 22:27         ` Michael Hope
2012-06-15  7:44           ` Eli Zaretskii
2012-06-15  7:36         ` Eli Zaretskii
2012-06-15 12:19           ` Joel Brobecker
2012-06-17 23:26           ` Michael Hope
2012-06-18  2:53             ` Eli Zaretskii
2012-06-21  4:20               ` Michael Hope
2012-06-21 16:22                 ` Eli Zaretskii
2012-06-21 22:36                   ` Michael Hope
2012-06-22  5:58                     ` Eli Zaretskii
2012-06-22 10:03                       ` Eli Zaretskii
2012-07-01 19:56                         ` Eli Zaretskii [this message]
2012-07-02  1:03                           ` Michael Hope
2012-07-03 16:23                             ` Eli Zaretskii
2012-07-11  5:14                             ` Terry Guo
     [not found]                             ` <000201cd5f24$1ff8e8c0$5feaba40$%guo@arm.com>
2012-07-12  7:57                               ` Eli Zaretskii
2012-06-15  7:29       ` Eli Zaretskii
2012-06-17 23:18         ` Michael Hope

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=83r4sv5k9h.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=brobecker@adacore.com \
    --cc=gdb-patches@sourceware.org \
    --cc=joseph@codesourcery.com \
    --cc=michael.hope@linaro.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox