From: Guinevere Larsen <guinevere@redhat.com>
To: "Maciej W. Rozycki" <macro@orcam.me.uk>
Cc: gdb-patches@sourceware.org
Subject: Re: [PATCH 1/2] gdb: introduce command "info architecture"
Date: Thu, 27 Nov 2025 16:51:51 -0300 [thread overview]
Message-ID: <67dc2271-4c15-42d1-bd1a-d77e2cd1e6df@redhat.com> (raw)
In-Reply-To: <alpine.DEB.2.21.2511071528570.25436@angie.orcam.me.uk>
[-- Attachment #1: Type: text/plain, Size: 1643 bytes --]
On 11/7/25 12:51 PM, Maciej W. Rozycki wrote:
> On Fri, 7 Nov 2025, Guinevere Larsen wrote:
>
>>> Or am I missing something? Of course adding filtering might be useful,
>>> especially if the list so large as in my quoted example, but the change
>>> description suggests there's no listing facility available.
>> Well, I wouldn't say we have a listing facility. using Set Architecture
>> without an argument may list them, but that is an error message, rather than a
>> command to do that, which is why I don't think it is a convenient way to tell
>> a user to check for support. You could achieve the same if you asked for
>> completion, but again, not a "listing facility" in the sense of telling a user
>> to check.
> We could reword output from `set architecture' so as not to suggest the
> use without an argument is invalid.
>
> In any case the listing facility must not be an `info' subcommand, as
> these are meant for showing the state of the debuggee,
I disagree, in my opinion, this new command would function basically
just like "info unwinder", giving information on how GDB is able to
interact with the inferior.
> while the set of
> architectures supported is a property of the given instance of GDB itself,
> and therefore suitable for the `set'/`show' commands. So if a new command
> it would have to be `show architecture list' or suchlike.
My understanding is that set/show commands are mostly used for when the
user could change something, and the point of this command is showing
the user the compilation options that were used, there's no changes that
a user could do.
--
Cheers,
Guinevere Larsen
It/she
[-- Attachment #2: Type: text/html, Size: 2561 bytes --]
next prev parent reply other threads:[~2025-11-27 19:52 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-11-06 19:45 [PATCH 0/2] Add warning if the native target is not supported Guinevere Larsen
2025-11-06 19:45 ` [PATCH 1/2] gdb: introduce command "info architecture" Guinevere Larsen
2025-11-06 23:24 ` Maciej W. Rozycki
2025-11-07 13:10 ` Guinevere Larsen
2025-11-07 15:51 ` Maciej W. Rozycki
2025-11-07 16:05 ` Hannes Domani
2025-11-08 2:02 ` Maciej W. Rozycki
2025-11-27 19:51 ` Guinevere Larsen [this message]
2025-11-27 20:00 ` Maciej W. Rozycki
2025-11-27 20:09 ` Guinevere Larsen
2025-11-28 9:20 ` Maciej W. Rozycki
2025-12-01 16:49 ` Guinevere Larsen
2025-11-07 7:46 ` Eli Zaretskii
2025-11-07 13:41 ` Guinevere Larsen
2025-11-07 14:23 ` Eli Zaretskii
2025-11-08 2:25 ` Maciej W. Rozycki
2025-11-27 20:01 ` Guinevere Larsen
2025-11-28 9:20 ` Maciej W. Rozycki
2025-11-06 19:45 ` [PATCH 2/2] gdb: add warning when no native target is available Guinevere Larsen
2025-11-07 7:48 ` Eli Zaretskii
2025-11-07 13:24 ` Guinevere Larsen
2025-11-07 14:21 ` Eli Zaretskii
2025-11-27 20:15 ` Guinevere Larsen
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=67dc2271-4c15-42d1-bd1a-d77e2cd1e6df@redhat.com \
--to=guinevere@redhat.com \
--cc=gdb-patches@sourceware.org \
--cc=macro@orcam.me.uk \
/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