[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] tests/qtest: Don't run the dbus-display-test without CONFIG_
From: |
Marc-André Lureau |
Subject: |
Re: [PATCH] tests/qtest: Don't run the dbus-display-test without CONFIG_VGA_PCI |
Date: |
Mon, 19 Feb 2024 18:00:32 +0400 |
Hi
On Mon, Feb 19, 2024 at 4:39 PM Thomas Huth <thuth@redhat.com> wrote:
>
> When compiling with "configure --without-default-devices", the
> dbus-display-test fails since it implicitly assumes that the
> machine comes with the standard VGA card. Thus add a check to
> meson.build to disable the test if the VGA card is not available.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
Not necessarily VGA, but a graphic or VC console. I am not sure how to
guess from the binary or compilation settings. Maybe it would be
simpler to check at run-time if /org/qemu/Display1/Console_0 exists. I
can work on a patch.
> ---
> tests/qtest/meson.build | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/qtest/meson.build b/tests/qtest/meson.build
> index 2b89e8634b..c8e6d7df40 100644
> --- a/tests/qtest/meson.build
> +++ b/tests/qtest/meson.build
> @@ -108,7 +108,7 @@ qtests_i386 = \
> 'numa-test'
> ]
>
> -if dbus_display
> +if dbus_display and config_all_devices.has_key('CONFIG_VGA_PCI')
> qtests_i386 += ['dbus-display-test']
> endif
>
> --
> 2.43.2
>