From: Doug Evans <dje@google.com>
To: Eli Zaretskii <eliz@gnu.org>
Cc: jan.kratochvil@redhat.com, gdb-patches@sourceware.org
Subject: Re: [patch, doc RFA] New command-line option -na.
Date: Wed, 15 Dec 2010 06:26:00 -0000 [thread overview]
Message-ID: <AANLkTingUcfwc8X=FPD467ZrHQ+WM+3Ov5TJY2pfbp4J@mail.gmail.com> (raw)
In-Reply-To: <E1PSkUR-0008Mm-3v@fencepost.gnu.org>
On Tue, Dec 14, 2010 at 10:05 PM, Eli Zaretskii <eliz@gnu.org> wrote:
> "Startup" is not well defined, from the user's POV. The manual uses
> that term loosely as something vaguely understandable by everyone.
> Wearing my user's hat, when I type "gdb core" or "gdb -p 1234",
> everything GDB does before I get my first prompt is "startup". That
> includes loading shared libraries (unless auto-solib-add is OFF),
> AFAIK.
Actually, give it a try.
$ gdb hello
(gdb) info shared
No shared libraries loaded at this time.
(gdb)
>> If you wish a different wording, can you provide one? Thanks.
>
> How about this one:
>
> If the command line specified a program to debug, or a process to
> attach to, or a core file, @value{GDBN} loads any auto-loaded
> scripts provided for the program or for its loaded shared libraries.
> @xref{Auto-Loading}.
Works for me, but I'll take out "or for its loaded shared libraries". Ok?
>
>> >> +@kindex set auto-load
>> >> +@item set auto-load [yes|no]
>> >> +Enable or disable the auto-loading of scripts.
>> >> +
>> >> +@kindex show auto-load
>> >> +@item show auto-load
>> >> +Show whether auto-loading is enabled or disabled.
>> >> @end table
>> >
>> > If we are going to leave this section inside the Python chapter, then
>> > we should say "auto-load Python scripts" etc. elsewhere, because this
>> > feature is limited to loading Python scripts, right?
>>
>> I like what it's in the patch.
>> There's no reason why one couldn't have non-python auto-loaded
>> scripts, and there's no reason at the moment for having a different
>> option to control them.
>> So I don't want "python" in the option name.
>> Having it the Python section of the manual is fine for now since
>> that's all we currently support.
>
> It's entirely unreasonable to have a command that is not limited to
> Python in a Python specific section. We could either move it out or
> say something like "Enable or disable auto-loading of Python scripts."
> (Note that I didn't ask to have "python" in the option's name.)
> Please choose one or the other; I can go with either one.
>
> I'm also open to other suggestions, if you (or someone else) have
> them.
I disagree. The option is only applicable to Python scripts today, so
why not leave it where it is?
We can always move it later.
But I'll go with mentioning python in the command's text.
next prev parent reply other threads:[~2010-12-15 6:26 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-12-11 0:47 Doug Evans
2010-12-11 8:16 ` Eli Zaretskii
2010-12-15 1:40 ` Doug Evans
2010-12-15 6:05 ` Eli Zaretskii
2010-12-15 6:26 ` Doug Evans [this message]
2010-12-15 6:58 ` Eli Zaretskii
2010-12-15 7:09 ` Doug Evans
2010-12-13 18:55 ` Jan Kratochvil
2010-12-13 19:28 ` Doug Evans
2010-12-13 19:36 ` Doug Evans
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='AANLkTingUcfwc8X=FPD467ZrHQ+WM+3Ov5TJY2pfbp4J@mail.gmail.com' \
--to=dje@google.com \
--cc=eliz@gnu.org \
--cc=gdb-patches@sourceware.org \
--cc=jan.kratochvil@redhat.com \
/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