From: Paolo Bonzini <pbonzini@redhat.com>
To: Thomas Huth <thuth@redhat.com>
Cc: "Marc-André Lureau" <marcandre.lureau@redhat.com>,
qemu-devel <qemu-devel@nongnu.org>
Subject: Re: [PATCH 0/3] Use meson_options.txt in the configure script
Date: Tue, 31 Aug 2021 14:49:11 +0200 [thread overview]
Message-ID: <CABgObfaYTGc8NZifmkWj2SWgcyC47LeiNRchVyEXd9mMQtQiZA@mail.gmail.com> (raw)
In-Reply-To: <48ccdd11-5836-5886-f928-95c960fb810a@redhat.com>
[-- Attachment #1: Type: text/plain, Size: 944 bytes --]
Il lun 30 ago 2021, 07:11 Thomas Huth <thuth@redhat.com> ha scritto:
> Paolo, why did your patch get stalled?
It just wasn't worthwhile at the time given how few meson options were
there. I missed 6.1 and was going to send it out again for 6.2.
The reason for using Perl was because there's no guarantee of knowing the
path to the Python interpreter until after --python is parsed, so there
would be a risk of not being able to give a full help. I don't like Perl
very much, but the difference in obscurity between Perl and Eric-level sh
is small. :)
Paolo
... my way of parsing is certainly
> way more fragile, but it's less complicated and seems to work as long as
> all
> the options are written in the same way in meson_options.txt (e.g. as long
> as nobody tries to use multi-line descriptions of the options there
> etc.)...
> so maybe if the additional Perl script was too much, this could be a good
> compromise?
>
> Thomas
>
>
[-- Attachment #2: Type: text/html, Size: 1594 bytes --]
prev parent reply other threads:[~2021-08-31 13:22 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-08-29 17:32 [PATCH 0/3] Use meson_options.txt in the configure script Thomas Huth
2021-08-29 17:32 ` [PATCH 1/3] configure: Add the possibility to read options from meson_options.txt Thomas Huth
2021-08-30 14:47 ` Eric Blake
2021-08-29 17:32 ` [PATCH 2/3] configure: Remove options that can be handled via meson_options.txt instead Thomas Huth
2021-08-30 15:06 ` Eric Blake
2021-08-30 15:33 ` Richard Henderson
2021-08-30 16:33 ` Thomas Huth
2021-08-29 17:32 ` [PATCH 3/3] configure: Get help text from meson_options.txt Thomas Huth
2021-08-30 15:30 ` Eric Blake
2021-08-30 16:48 ` Thomas Huth
2021-08-29 21:22 ` [PATCH 0/3] Use meson_options.txt in the configure script Marc-André Lureau
2021-08-30 5:11 ` Thomas Huth
2021-08-30 9:12 ` Philippe Mathieu-Daudé
2021-08-30 9:21 ` Peter Maydell
2021-08-31 12:49 ` Paolo Bonzini [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=CABgObfaYTGc8NZifmkWj2SWgcyC47LeiNRchVyEXd9mMQtQiZA@mail.gmail.com \
--to=pbonzini@redhat.com \
--cc=marcandre.lureau@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=thuth@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;
as well as URLs for NNTP newsgroup(s).