[v3] add build option to disable audio subsystem

[PATCH v3 08/11] tests/audio: do not compile if audio is disabled

Sergei Heifetz posted 11 patches 3 weeks, 1 day ago

Diff against v1 v2
Download mbox

Maintainers: Paolo Bonzini <pbonzini@redhat.com>, Gerd Hoffmann <kraxel@redhat.com>, "Marc-André Lureau" <marcandre.lureau@redhat.com>, "Dr. David Alan Gilbert" <dave@treblig.org>, Marcel Apfelbaum <marcel.apfelbaum@gmail.com>, "Philippe Mathieu-Daudé" <philmd@linaro.org>, Yanan Wang <wangyanan55@huawei.com>, Zhao Liu <zhao1.liu@intel.com>, "Daniel P. Berrangé" <berrange@redhat.com>, "Michael S. Tsirkin" <mst@redhat.com>, Richard Henderson <richard.henderson@linaro.org>, Eric Blake <eblake@redhat.com>, Markus Armbruster <armbru@redhat.com>, "Alex Bennée" <alex.bennee@linaro.org>, Fabiano Rosas <farosas@suse.de>, Laurent Vivier <lvivier@redhat.com>

Expand all Fold all

[PATCH v3 08/11] tests/audio: do not compile if audio is disabled

Posted by Sergei Heifetz 3 weeks, 1 day ago

If QEMU is configured with --disable-audio, do not compile the audio
tests.

Signed-off-by: Sergei Heifetz <heifetz@yandex-team.com>
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
---
 tests/audio/meson.build | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/audio/meson.build b/tests/audio/meson.build
index 84754bde221..97c1d623bbd 100644
--- a/tests/audio/meson.build
+++ b/tests/audio/meson.build
@@ -1,6 +1,6 @@
 # SPDX-License-Identifier: GPL-2.0-or-later
 
-if not have_system
+if not have_audio
   subdir_done()
 endif
 
-- 
2.34.1

Re: [PATCH v3 08/11] tests/audio: do not compile if audio is disabled

Posted by Philippe Mathieu-Daudé 5 days, 20 hours ago

On 15/3/26 21:16, Sergei Heifetz wrote:
> If QEMU is configured with --disable-audio, do not compile the audio
> tests.
> 
> Signed-off-by: Sergei Heifetz <heifetz@yandex-team.com>
> Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
> ---
>   tests/audio/meson.build | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/audio/meson.build b/tests/audio/meson.build
> index 84754bde221..97c1d623bbd 100644
> --- a/tests/audio/meson.build
> +++ b/tests/audio/meson.build
> @@ -1,6 +1,6 @@
>   # SPDX-License-Identifier: GPL-2.0-or-later
>   
> -if not have_system
> +if not have_audio
>     subdir_done()
>   endif
>   

Squash with the meson change in patch #1?

Re: [PATCH v3 08/11] tests/audio: do not compile if audio is disabled

Posted by Sergei Heifetz 5 days, 9 hours ago

On Wed Apr 1, 2026 at 3:10 PM +05, Philippe Mathieu-Daudé wrote:
> On 15/3/26 21:16, Sergei Heifetz wrote:
>> If QEMU is configured with --disable-audio, do not compile the audio
>> tests.
>> 
>> Signed-off-by: Sergei Heifetz <heifetz@yandex-team.com>
>> Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
>> ---
>>   tests/audio/meson.build | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>> 
>> diff --git a/tests/audio/meson.build b/tests/audio/meson.build
>> index 84754bde221..97c1d623bbd 100644
>> --- a/tests/audio/meson.build
>> +++ b/tests/audio/meson.build
>> @@ -1,6 +1,6 @@
>>   # SPDX-License-Identifier: GPL-2.0-or-later
>>   
>> -if not have_system
>> +if not have_audio
>>     subdir_done()
>>   endif
>>   
>
> Squash with the meson change in patch #1?

Yes, sure.