[PATCH 0/3] meson.build: Group some entries in separate summary sections

[PATCH 0/3] meson.build: Group some entries in separate summary sections

[Thomas Huth]

For the average user, it is likely quite difficult which library is
responsible for different features that QEMU supports. Let's make
it a little bit easier for them and put some libraries into separate
groups in the summary output of meson.

Thomas Huth (3):
  meson.build: Group the UI entries in a separate summary section
  meson.build: Group the network backend entries in a separate summary
    section
  meson.build: Group the audio backend entries in a separate summary
    section

 meson.build | 48 +++++++++++++++++++++++++++++++-----------------
 1 file changed, 31 insertions(+), 17 deletions(-)

-- 
2.31.1

[PATCH 1/3] meson.build: Group the UI entries in a separate summary section

[Thomas Huth]

Let's make it easier for the users to spot UI-related entries in
the summary of the meson output.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 meson.build | 35 +++++++++++++++++++++--------------
 1 file changed, 21 insertions(+), 14 deletions(-)

diff --git a/meson.build b/meson.build
index a61d3e9b06..4a20a2e712 100644
--- a/meson.build
+++ b/meson.build
@@ -4243,32 +4243,44 @@ summary_info += {'rng-none':          get_option('rng_none')}
 summary_info += {'Linux keyring':     have_keyring}
 summary(summary_info, bool_yn: true, section: 'Crypto')
 
-# Libraries
+# UI
 summary_info = {}
 if targetos == 'darwin'
   summary_info += {'Cocoa support':           cocoa}
-  summary_info += {'vmnet.framework support': vmnet}
 endif
 summary_info += {'SDL support':       sdl}
 summary_info += {'SDL image support': sdl_image}
 summary_info += {'GTK support':       gtk}
 summary_info += {'pixman':            pixman}
 summary_info += {'VTE support':       vte}
+summary_info += {'PNG support':       png}
+summary_info += {'VNC support':       vnc}
+if vnc.found()
+  summary_info += {'VNC SASL support':  sasl}
+  summary_info += {'VNC JPEG support':  jpeg}
+endif
+summary_info += {'spice protocol support': spice_protocol}
+if spice_protocol.found()
+  summary_info += {'  spice server support': spice}
+endif
+summary_info += {'curses support':    curses}
+summary_info += {'brlapi support':    brlapi}
+summary(summary_info, bool_yn: true, section: 'User interface')
+
+# Libraries
+summary_info = {}
+if targetos == 'darwin'
+  summary_info += {'vmnet.framework support': vmnet}
+endif
+summary_info = {}
 summary_info += {'slirp support':     slirp}
 summary_info += {'libtasn1':          tasn1}
 summary_info += {'PAM':               pam}
 summary_info += {'iconv support':     iconv}
-summary_info += {'curses support':    curses}
 summary_info += {'virgl support':     virgl}
 summary_info += {'blkio support':     blkio}
 summary_info += {'curl support':      curl}
 summary_info += {'Multipath support': mpathpersist}
-summary_info += {'PNG support':       png}
-summary_info += {'VNC support':       vnc}
-if vnc.found()
-  summary_info += {'VNC SASL support':  sasl}
-  summary_info += {'VNC JPEG support':  jpeg}
-endif
 if targetos not in ['darwin', 'haiku', 'windows']
   summary_info += {'OSS support':     oss}
   summary_info += {'sndio support':   sndio}
@@ -4283,7 +4295,6 @@ if targetos == 'linux'
 endif
 summary_info += {'Pipewire support':   pipewire}
 summary_info += {'JACK support':      jack}
-summary_info += {'brlapi support':    brlapi}
 summary_info += {'vde support':       vde}
 summary_info += {'netmap support':    have_netmap}
 summary_info += {'l2tpv3 support':    have_l2tpv3}
@@ -4295,10 +4306,6 @@ summary_info += {'PVRDMA support':    have_pvrdma}
 summary_info += {'fdt support':       fdt_opt == 'disabled' ? false : fdt_opt}
 summary_info += {'libcap-ng support': libcap_ng}
 summary_info += {'bpf support':       libbpf}
-summary_info += {'spice protocol support': spice_protocol}
-if spice_protocol.found()
-  summary_info += {'  spice server support': spice}
-endif
 summary_info += {'rbd support':       rbd}
 summary_info += {'smartcard support': cacard}
 summary_info += {'U2F support':       u2f}
-- 
2.31.1

[PATCH 2/3] meson.build: Group the network backend entries in a separate summary section

[Thomas Huth]

Let's make it easier for the users to spot network-related entries
in the summary of the meson output.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 meson.build | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/meson.build b/meson.build
index 4a20a2e712..c64ad3c365 100644
--- a/meson.build
+++ b/meson.build
@@ -4267,13 +4267,19 @@ summary_info += {'curses support':    curses}
 summary_info += {'brlapi support':    brlapi}
 summary(summary_info, bool_yn: true, section: 'User interface')
 
-# Libraries
+# Network backends
 summary_info = {}
 if targetos == 'darwin'
   summary_info += {'vmnet.framework support': vmnet}
 endif
-summary_info = {}
 summary_info += {'slirp support':     slirp}
+summary_info += {'vde support':       vde}
+summary_info += {'netmap support':    have_netmap}
+summary_info += {'l2tpv3 support':    have_l2tpv3}
+summary(summary_info, bool_yn: true, section: 'Network backends')
+
+# Libraries
+summary_info = {}
 summary_info += {'libtasn1':          tasn1}
 summary_info += {'PAM':               pam}
 summary_info += {'iconv support':     iconv}
@@ -4295,9 +4301,6 @@ if targetos == 'linux'
 endif
 summary_info += {'Pipewire support':   pipewire}
 summary_info += {'JACK support':      jack}
-summary_info += {'vde support':       vde}
-summary_info += {'netmap support':    have_netmap}
-summary_info += {'l2tpv3 support':    have_l2tpv3}
 summary_info += {'Linux AIO support': libaio}
 summary_info += {'Linux io_uring support': linux_io_uring}
 summary_info += {'ATTR/XATTR support': libattr}
-- 
2.31.1

[PATCH 3/3] meson.build: Group the audio backend entries in a separate summary section

[Thomas Huth]

Let's make it easier for the users to spot audio-related entries
in the summary of the meson output.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 meson.build | 32 ++++++++++++++++++--------------
 1 file changed, 18 insertions(+), 14 deletions(-)

diff --git a/meson.build b/meson.build
index c64ad3c365..365c6d60a5 100644
--- a/meson.build
+++ b/meson.build
@@ -4267,6 +4267,24 @@ summary_info += {'curses support':    curses}
 summary_info += {'brlapi support':    brlapi}
 summary(summary_info, bool_yn: true, section: 'User interface')
 
+# Audio backends
+summary_info = {}
+if targetos not in ['darwin', 'haiku', 'windows']
+  summary_info += {'OSS support':     oss}
+  summary_info += {'sndio support':   sndio}
+elif targetos == 'darwin'
+  summary_info += {'CoreAudio support': coreaudio}
+elif targetos == 'windows'
+  summary_info += {'DirectSound support': dsound}
+endif
+if targetos == 'linux'
+  summary_info += {'ALSA support':    alsa}
+  summary_info += {'PulseAudio support': pulse}
+endif
+summary_info += {'Pipewire support':  pipewire}
+summary_info += {'JACK support':      jack}
+summary(summary_info, bool_yn: true, section: 'Audio backends')
+
 # Network backends
 summary_info = {}
 if targetos == 'darwin'
@@ -4287,20 +4305,6 @@ summary_info += {'virgl support':     virgl}
 summary_info += {'blkio support':     blkio}
 summary_info += {'curl support':      curl}
 summary_info += {'Multipath support': mpathpersist}
-if targetos not in ['darwin', 'haiku', 'windows']
-  summary_info += {'OSS support':     oss}
-  summary_info += {'sndio support':   sndio}
-elif targetos == 'darwin'
-  summary_info += {'CoreAudio support': coreaudio}
-elif targetos == 'windows'
-  summary_info += {'DirectSound support': dsound}
-endif
-if targetos == 'linux'
-  summary_info += {'ALSA support':    alsa}
-  summary_info += {'PulseAudio support': pulse}
-endif
-summary_info += {'Pipewire support':   pipewire}
-summary_info += {'JACK support':      jack}
 summary_info += {'Linux AIO support': libaio}
 summary_info += {'Linux io_uring support': linux_io_uring}
 summary_info += {'ATTR/XATTR support': libattr}
-- 
2.31.1

Re: [PATCH 1/3] meson.build: Group the UI entries in a separate summary section

[Philippe Mathieu-Daudé]

On 2/6/23 19:18, Thomas Huth wrote:
> Let's make it easier for the users to spot UI-related entries in
> the summary of the meson output.
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>   meson.build | 35 +++++++++++++++++++++--------------
>   1 file changed, 21 insertions(+), 14 deletions(-)
> 
> diff --git a/meson.build b/meson.build
> index a61d3e9b06..4a20a2e712 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -4243,32 +4243,44 @@ summary_info += {'rng-none':          get_option('rng_none')}
>   summary_info += {'Linux keyring':     have_keyring}
>   summary(summary_info, bool_yn: true, section: 'Crypto')
>   
> -# Libraries
> +# UI
>   summary_info = {}
>   if targetos == 'darwin'
>     summary_info += {'Cocoa support':           cocoa}
> -  summary_info += {'vmnet.framework support': vmnet}
>   endif
>   summary_info += {'SDL support':       sdl}
>   summary_info += {'SDL image support': sdl_image}
>   summary_info += {'GTK support':       gtk}
>   summary_info += {'pixman':            pixman}
>   summary_info += {'VTE support':       vte}
> +summary_info += {'PNG support':       png}
> +summary_info += {'VNC support':       vnc}
> +if vnc.found()
> +  summary_info += {'VNC SASL support':  sasl}
> +  summary_info += {'VNC JPEG support':  jpeg}
> +endif
> +summary_info += {'spice protocol support': spice_protocol}
> +if spice_protocol.found()
> +  summary_info += {'  spice server support': spice}
> +endif
> +summary_info += {'curses support':    curses}
> +summary_info += {'brlapi support':    brlapi}
> +summary(summary_info, bool_yn: true, section: 'User interface')
> +
> +# Libraries
> +summary_info = {}
> +if targetos == 'darwin'
> +  summary_info += {'vmnet.framework support': vmnet}
> +endif
> +summary_info = {}

Conditional to dropping the previous line:
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>

>   summary_info += {'slirp support':     slirp}
>   summary_info += {'libtasn1':          tasn1}
>   summary_info += {'PAM':               pam}
>   summary_info += {'iconv support':     iconv}
> -summary_info += {'curses support':    curses}
>   summary_info += {'virgl support':     virgl}
>   summary_info += {'blkio support':     blkio}
>   summary_info += {'curl support':      curl}
>   summary_info += {'Multipath support': mpathpersist}
> -summary_info += {'PNG support':       png}
> -summary_info += {'VNC support':       vnc}
> -if vnc.found()
> -  summary_info += {'VNC SASL support':  sasl}
> -  summary_info += {'VNC JPEG support':  jpeg}
> -endif
>   if targetos not in ['darwin', 'haiku', 'windows']
>     summary_info += {'OSS support':     oss}
>     summary_info += {'sndio support':   sndio}
> @@ -4283,7 +4295,6 @@ if targetos == 'linux'
>   endif
>   summary_info += {'Pipewire support':   pipewire}
>   summary_info += {'JACK support':      jack}
> -summary_info += {'brlapi support':    brlapi}
>   summary_info += {'vde support':       vde}
>   summary_info += {'netmap support':    have_netmap}
>   summary_info += {'l2tpv3 support':    have_l2tpv3}
> @@ -4295,10 +4306,6 @@ summary_info += {'PVRDMA support':    have_pvrdma}
>   summary_info += {'fdt support':       fdt_opt == 'disabled' ? false : fdt_opt}
>   summary_info += {'libcap-ng support': libcap_ng}
>   summary_info += {'bpf support':       libbpf}
> -summary_info += {'spice protocol support': spice_protocol}
> -if spice_protocol.found()
> -  summary_info += {'  spice server support': spice}
> -endif
>   summary_info += {'rbd support':       rbd}
>   summary_info += {'smartcard support': cacard}
>   summary_info += {'U2F support':       u2f}

Re: [PATCH 2/3] meson.build: Group the network backend entries in a separate summary section

[Philippe Mathieu-Daudé]

On 2/6/23 19:18, Thomas Huth wrote:
> Let's make it easier for the users to spot network-related entries
> in the summary of the meson output.
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>   meson.build | 13 ++++++++-----
>   1 file changed, 8 insertions(+), 5 deletions(-)
> 
> diff --git a/meson.build b/meson.build
> index 4a20a2e712..c64ad3c365 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -4267,13 +4267,19 @@ summary_info += {'curses support':    curses}
>   summary_info += {'brlapi support':    brlapi}
>   summary(summary_info, bool_yn: true, section: 'User interface')
>   
> -# Libraries
> +# Network backends
>   summary_info = {}
>   if targetos == 'darwin'
>     summary_info += {'vmnet.framework support': vmnet}
>   endif
> -summary_info = {}

Ah, this should be squashed in the previous patch.

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>

>   summary_info += {'slirp support':     slirp}
> +summary_info += {'vde support':       vde}
> +summary_info += {'netmap support':    have_netmap}
> +summary_info += {'l2tpv3 support':    have_l2tpv3}
> +summary(summary_info, bool_yn: true, section: 'Network backends')
> +
> +# Libraries
> +summary_info = {}
>   summary_info += {'libtasn1':          tasn1}
>   summary_info += {'PAM':               pam}
>   summary_info += {'iconv support':     iconv}
> @@ -4295,9 +4301,6 @@ if targetos == 'linux'
>   endif
>   summary_info += {'Pipewire support':   pipewire}
>   summary_info += {'JACK support':      jack}
> -summary_info += {'vde support':       vde}
> -summary_info += {'netmap support':    have_netmap}
> -summary_info += {'l2tpv3 support':    have_l2tpv3}
>   summary_info += {'Linux AIO support': libaio}
>   summary_info += {'Linux io_uring support': linux_io_uring}
>   summary_info += {'ATTR/XATTR support': libattr}

Re: [PATCH 3/3] meson.build: Group the audio backend entries in a separate summary section

[Philippe Mathieu-Daudé]

On 2/6/23 19:18, Thomas Huth wrote:
> Let's make it easier for the users to spot audio-related entries
> in the summary of the meson output.
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>   meson.build | 32 ++++++++++++++++++--------------
>   1 file changed, 18 insertions(+), 14 deletions(-)

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>

Re: [PATCH 0/3] meson.build: Group some entries in separate summary sections

[Richard Henderson]

On 6/2/23 10:18, Thomas Huth wrote:
> For the average user, it is likely quite difficult which library is
> responsible for different features that QEMU supports. Let's make
> it a little bit easier for them and put some libraries into separate
> groups in the summary output of meson.
> 
> Thomas Huth (3):
>    meson.build: Group the UI entries in a separate summary section
>    meson.build: Group the network backend entries in a separate summary
>      section
>    meson.build: Group the audio backend entries in a separate summary
>      section
> 
>   meson.build | 48 +++++++++++++++++++++++++++++++-----------------
>   1 file changed, 31 insertions(+), 17 deletions(-)
> 

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~

Re: [PATCH 2/3] meson.build: Group the network backend entries in a separate summary section

[Thomas Huth]

On 02/06/2023 22.03, Philippe Mathieu-Daudé wrote:
> On 2/6/23 19:18, Thomas Huth wrote:
>> Let's make it easier for the users to spot network-related entries
>> in the summary of the meson output.
>>
>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>> ---
>>   meson.build | 13 ++++++++-----
>>   1 file changed, 8 insertions(+), 5 deletions(-)
>>
>> diff --git a/meson.build b/meson.build
>> index 4a20a2e712..c64ad3c365 100644
>> --- a/meson.build
>> +++ b/meson.build
>> @@ -4267,13 +4267,19 @@ summary_info += {'curses support':    curses}
>>   summary_info += {'brlapi support':    brlapi}
>>   summary(summary_info, bool_yn: true, section: 'User interface')
>> -# Libraries
>> +# Network backends
>>   summary_info = {}
>>   if targetos == 'darwin'
>>     summary_info += {'vmnet.framework support': vmnet}
>>   endif
>> -summary_info = {}
> 
> Ah, this should be squashed in the previous patch.

Drat, thanks for catching it!

> Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>

Thanks!

  Thomas

>>   summary_info += {'slirp support':     slirp}
>> +summary_info += {'vde support':       vde}
>> +summary_info += {'netmap support':    have_netmap}
>> +summary_info += {'l2tpv3 support':    have_l2tpv3}
>> +summary(summary_info, bool_yn: true, section: 'Network backends')
>> +
>> +# Libraries
>> +summary_info = {}
>>   summary_info += {'libtasn1':          tasn1}
>>   summary_info += {'PAM':               pam}
>>   summary_info += {'iconv support':     iconv}
>> @@ -4295,9 +4301,6 @@ if targetos == 'linux'
>>   endif
>>   summary_info += {'Pipewire support':   pipewire}
>>   summary_info += {'JACK support':      jack}
>> -summary_info += {'vde support':       vde}
>> -summary_info += {'netmap support':    have_netmap}
>> -summary_info += {'l2tpv3 support':    have_l2tpv3}
>>   summary_info += {'Linux AIO support': libaio}
>>   summary_info += {'Linux io_uring support': linux_io_uring}
>>   summary_info += {'ATTR/XATTR support': libattr}
> 

Re: [PATCH 0/3] meson.build: Group some entries in separate summary sections

[Paolo Bonzini]

Queued, thanks.

Paolo