From: Peter Maydell <peter.maydell@linaro.org>
To: Stefan Weil <weil@mail.berlios.de>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH] configure: Support --target-list=? to list available targets
Date: Fri, 22 Apr 2011 01:50:53 +0100 [thread overview]
Message-ID: <BANLkTimcFsPPDD6r2RKOrwrF3KT5Nna8vQ@mail.gmail.com> (raw)
In-Reply-To: <4DB06D28.1040007@mail.berlios.de>
On 21 April 2011 18:45, Stefan Weil <weil@mail.berlios.de> wrote:
> Am 21.04.2011 15:10, schrieb Peter Maydell:
>> Add support for getting configure to print the list of all targets
>> that can be built, via the option '--target-list=?'.
> The qemu executable supports this use pattern (-cpu ?),
> but it's unusual for configure.
>
> Other options (--audio-drv-list=LIST, --audio-card-list=LIST)
> show the available values in the help message (configure --help).
>
> What about a similar help text for --target-list? Like this:
>
> --target-list=LIST set target list []
> Available targets: i386-softmmu ...
> i386-linux-user ...
The "[]" here is wrong, incidentally, because the bit in [] is
supposed to be the default value, which for the target list is
not "build no targets" but "build all targets". So we could
either say:
--target-list=LIST set target list (default: build everything)
Available targets: i386-softmmu ...
i386-linux-user ...
or
--target-list=LIST set target list [i386-softmmu ...
i386-linux-user ...]
I vaguely prefer the first but have no strong feelings.
The other issue is that the target list is vastly longer than
any of the existing lists in the help, so it probably needs
something to automatically format it so it doesn't mess up
the formatting of the --help message.
(We could perhaps support --option=? for all the options that
take a list, in addition to giving the supported values in
--help, but maybe that's overkill.)
-- PMM
prev parent reply other threads:[~2011-04-22 0:50 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-21 13:10 [Qemu-devel] [PATCH] configure: Support --target-list=? to list available targets Peter Maydell
2011-04-21 17:45 ` Stefan Weil
2011-04-22 0:50 ` Peter Maydell [this message]
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=BANLkTimcFsPPDD6r2RKOrwrF3KT5Nna8vQ@mail.gmail.com \
--to=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=weil@mail.berlios.de \
/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;
as well as URLs for NNTP newsgroup(s).