Re: [PATCH 19/30] configure, meson: move --enable-debug-info to Meson

[Philippe Mathieu-Daudé]

On 9/12/22 12:23, Paolo Bonzini wrote:
> Older versions of Meson had an unclear description of the -Ddebug option,
> but this is fixed in 0.63.0 that is required now.
>
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
>   configure                     | 5 -----
>   scripts/meson-buildoptions.py | 2 ++
>   scripts/meson-buildoptions.sh | 3 +++
>   3 files changed, 5 insertions(+), 5 deletions(-)


> diff --git a/scripts/meson-buildoptions.sh b/scripts/meson-buildoptions.sh
> index f91797741eef..cb277347bb18 100644
> --- a/scripts/meson-buildoptions.sh
> +++ b/scripts/meson-buildoptions.sh

# This file is generated by meson-buildoptions.py, do not edit!

> @@ -10,6 +10,7 @@ meson_options_help() {
>     printf "%s\n" '                           affects only QEMU, not tools like 
> qemu-img)'
>     printf "%s\n" '  --datadir=VALUE          Data file directory [share]'
>     printf "%s\n" '  --disable-coroutine-pool coroutine freelist (better 
> performance)'
> +  printf "%s\n" '  --disable-debug-info     Enable debug symbols and other 
> information'

We should get '--enable-debug-info' here, ...

>     printf "%s\n" '  --disable-install-blobs  install provided firmware blobs'

... and here. Do we have a bug in meson-buildoptions.py?

>     printf "%s\n" '  --docdir=VALUE           Base directory for documentation 
> installation'
>     printf "%s\n" '                           (can be empty) [share/doc]'