Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: "Terry Guo" <terry.guo@arm.com>
To: "Eli Zaretskii" <eliz@gnu.org>
Cc: <brobecker@adacore.com>,	<joseph@codesourcery.com>,
	<gdb-patches@sourceware.org>,
	"'Michael Hope'" <michael.hope@linaro.org>
Subject: RE: [PATCH, doc]: Rename Index node to prevent file collision
Date: Wed, 11 Jul 2012 05:14:00 -0000	[thread overview]
Message-ID: <000201cd5f24$1ff8e8c0$5feaba40$@guo@arm.com> (raw)
In-Reply-To: <CANLjY-kzUB36rFtC64n-W0fdGQOh1TPoUBs9H0PNWWbjjbOOyQ@mail.gmail.com>

Hi Eli,

When "make html" for gdbint.texi on case-insensitive Mac OS X, we still meet
index.html and Index.html issue. Could you please also apply same fix to
gdbint.texi? Thanks.

BR,
Terry

> -----Original Message-----
> From: gdb-patches-owner@sourceware.org [mailto:gdb-patches-
> owner@sourceware.org] On Behalf Of Michael Hope
> Sent: Monday, July 02, 2012 9:03 AM
> To: Eli Zaretskii
> Cc: brobecker@adacore.com; joseph@codesourcery.com; gdb-
> patches@sourceware.org
> Subject: Re: [PATCH, doc]: Rename Index node to prevent file collision
> 
> On 2 July 2012 07:55, Eli Zaretskii <eliz@gnu.org> wrote:
> > Ping!  Should I commit this?
> 
> It fixes my issue,
> 
> -- Michael
> 
> >> 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:
> >>




  parent reply	other threads:[~2012-07-11  5:14 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
2012-07-02  1:03                           ` Michael Hope
2012-07-03 16:23                             ` Eli Zaretskii
2012-07-11  5:14                             ` Terry Guo [this message]
     [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='000201cd5f24$1ff8e8c0$5feaba40$@guo@arm.com' \
    --to=terry.guo@arm.com \
    --cc=brobecker@adacore.com \
    --cc=eliz@gnu.org \
    --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