meson.build: Put the D-Bus summary into the UI section

[PATCH] meson.build: Put the D-Bus summary into the UI section

Thomas Huth posted 1 patch 10 months, 2 weeks ago

Download mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20250325055125.253669-1-thuth@redhat.com

Maintainers: Paolo Bonzini <pbonzini@redhat.com>, "Marc-André Lureau" <marcandre.lureau@redhat.com>, "Daniel P. Berrangé" <berrange@redhat.com>, "Philippe Mathieu-Daudé" <philmd@linaro.org>

meson.build | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

Expand all Fold all

[PATCH] meson.build: Put the D-Bus summary into the UI section

Posted by Thomas Huth 10 months, 2 weeks ago

From: Thomas Huth <thuth@redhat.com>

We've got a dedicated section for UI options nowadays, so the
D-Bus display should get reported here, too.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 meson.build | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meson.build b/meson.build
index 41f68d38069..205fb43bf7b 100644
--- a/meson.build
+++ b/meson.build
@@ -4559,7 +4559,6 @@ summary_info += {'Trace backends':    ','.join(get_option('trace_backends'))}
 if 'simple' in get_option('trace_backends')
   summary_info += {'Trace output file': get_option('trace_file') + '-<pid>'}
 endif
-summary_info += {'D-Bus display':     dbus_display}
 summary_info += {'QOM debugging':     get_option('qom_cast_debug')}
 summary_info += {'Relocatable install': get_option('relocatable')}
 summary_info += {'vhost-kernel support': have_vhost_kernel}
@@ -4742,6 +4741,7 @@ summary_info = {}
 if host_os == 'darwin'
   summary_info += {'Cocoa support':           cocoa}
 endif
+summary_info += {'D-Bus display':     dbus_display}
 summary_info += {'SDL support':       sdl}
 summary_info += {'SDL image support': sdl_image}
 summary_info += {'GTK support':       gtk}
-- 
2.49.0

Re: [PATCH] meson.build: Put the D-Bus summary into the UI section

Posted by Marc-André Lureau 10 months, 2 weeks ago

On Tue, Mar 25, 2025 at 9:51 AM Thomas Huth <thuth@redhat.com> wrote:
>
> From: Thomas Huth <thuth@redhat.com>
>
> We've got a dedicated section for UI options nowadays, so the
> D-Bus display should get reported here, too.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>

Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>

> ---
>  meson.build | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meson.build b/meson.build
> index 41f68d38069..205fb43bf7b 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -4559,7 +4559,6 @@ summary_info += {'Trace backends':    ','.join(get_option('trace_backends'))}
>  if 'simple' in get_option('trace_backends')
>    summary_info += {'Trace output file': get_option('trace_file') + '-<pid>'}
>  endif
> -summary_info += {'D-Bus display':     dbus_display}
>  summary_info += {'QOM debugging':     get_option('qom_cast_debug')}
>  summary_info += {'Relocatable install': get_option('relocatable')}
>  summary_info += {'vhost-kernel support': have_vhost_kernel}
> @@ -4742,6 +4741,7 @@ summary_info = {}
>  if host_os == 'darwin'
>    summary_info += {'Cocoa support':           cocoa}
>  endif
> +summary_info += {'D-Bus display':     dbus_display}
>  summary_info += {'SDL support':       sdl}
>  summary_info += {'SDL image support': sdl_image}
>  summary_info += {'GTK support':       gtk}
> --
> 2.49.0
>