Re: [PATCH 0/3] Use meson_options.txt in the configure script

2021-08-31 Thread Paolo Bonzini
Il lun 30 ago 2021, 07:11 Thomas Huth 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 knowin

Re: [PATCH 0/3] Use meson_options.txt in the configure script

2021-08-30 Thread Peter Maydell
On Mon, 30 Aug 2021 at 10:14, Philippe Mathieu-Daudé wrote: > IIRC while Perl is perfect for parsing, the maintenance cost is too > high. The git-forge / meson switch proved next-gen contributors are > more at ease with Python. That seems a bit strong to me. But I would say that for code where mo

Re: [PATCH 0/3] Use meson_options.txt in the configure script

2021-08-30 Thread Philippe Mathieu-Daudé
On 8/30/21 7:11 AM, Thomas Huth wrote: > On 29/08/2021 23.22, Marc-André Lureau wrote: >> Hi Thomas >> >> On Sun, Aug 29, 2021 at 9:32 PM Thomas Huth > > wrote: >> >>     It's cumbersome to maintain the build options twice, one time in the >>     configure script and one ti

Re: [PATCH 0/3] Use meson_options.txt in the configure script

2021-08-29 Thread Thomas Huth
On 29/08/2021 23.22, Marc-André Lureau wrote: Hi Thomas On Sun, Aug 29, 2021 at 9:32 PM Thomas Huth > wrote: It's cumbersome to maintain the build options twice, one time in the configure script and one time in meson_options.txt. Thus let's add some logic t

Re: [PATCH 0/3] Use meson_options.txt in the configure script

2021-08-29 Thread Marc-André Lureau
Hi Thomas On Sun, Aug 29, 2021 at 9:32 PM Thomas Huth wrote: > It's cumbersome to maintain the build options twice, one time in the > configure script and one time in meson_options.txt. Thus let's add some > logic to the configure script to look at the meson_options.txt file > instead of handlin

[PATCH 0/3] Use meson_options.txt in the configure script

2021-08-29 Thread Thomas Huth
It's cumbersome to maintain the build options twice, one time in the configure script and one time in meson_options.txt. Thus let's add some logic to the configure script to look at the meson_options.txt file instead of handling every option twice. Thomas Huth (3): configure: Add the possibility