Re: [PATCH 8/8] configure: automatically parse command line for meson -D options

2021-01-23 Thread Paolo Bonzini
On 23/01/21 04:30, 罗勇刚(Yonggang Luo) wrote: > We could and I did it in the first version. However it's ugly that the user has to use --python on some setups in order to get a full help message.   Yeap, but finally configure should gone, so I think --python are acceptable by user, just need

Re: [PATCH 8/8] configure: automatically parse command line for meson -D options

2021-01-22 Thread Yonggang Luo
On Sat, Jan 23, 2021 at 4:44 AM Paolo Bonzini wrote: > > > > Il ven 22 gen 2021, 09:00 罗勇刚(Yonggang Luo) ha scritto: >> >> Hi Paolo, as python and meson are required dependencies to building qemu now, >> can we detecting python/meson at the very begining of configure, >> even before the --help

Re: [PATCH 8/8] configure: automatically parse command line for meson -D options

2021-01-22 Thread Paolo Bonzini
Il ven 22 gen 2021, 09:00 罗勇刚(Yonggang Luo) ha scritto: > Hi Paolo, as python and meson are required dependencies to building qemu > now, > can we detecting python/meson at the very begining of configure, > even before the --help parameter. > We could and I did it in the first version. However

Re: [PATCH 8/8] configure: automatically parse command line for meson -D options

2021-01-22 Thread Yonggang Luo
Hi Paolo, as python and meson are required dependencies to building qemu now, can we detecting python/meson at the very begining of configure, even before the --help parameter. On Wed, Jan 13, 2021 at 6:08 AM Paolo Bonzini wrote: > > On 13/01/21 11:31, Daniel P. Berrangé wrote: > >>

Re: [PATCH 8/8] configure: automatically parse command line for meson -D options

2021-01-13 Thread Paolo Bonzini
On 13/01/21 11:31, Daniel P. Berrangé wrote: meson-buildoptions.json | 717 I'm not a fan of seeing this file introduced as it has significant overlap with meson_options.txt.I feel like the latter has enough information present to do an acceptable

Re: [PATCH 8/8] configure: automatically parse command line for meson -D options

2021-01-13 Thread Paolo Bonzini
On 13/01/21 11:31, Daniel P. Berrangé wrote: - because we need to run the script to generate the full help, we cannot rely on the user supplying the path to a Python interpreter with --python. For this reason, the script is written in Perl. Perl 5 is universally available as

Re: [PATCH 8/8] configure: automatically parse command line for meson -D options

2021-01-13 Thread Daniel P . Berrangé
On Thu, Jan 07, 2021 at 03:00:39PM +0100, Paolo Bonzini wrote: > Right now meson_options.txt lists almost 60 options. Each option > needs code in configure to parse it and pass the option down to Meson as > a -D command-line argument; in addition the default must be duplicated > between configure

[PATCH 8/8] configure: automatically parse command line for meson -D options

2021-01-07 Thread Paolo Bonzini
Right now meson_options.txt lists almost 60 options. Each option needs code in configure to parse it and pass the option down to Meson as a -D command-line argument; in addition the default must be duplicated between configure and meson_options.txt. This series tries to remove the code