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:
> >>
next prev 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