Re: [PATCH v2 00/12] buildsys: Do not build various objects if not necessary

[Alex Bennée]

Philippe Mathieu-Daudé <philmd@redhat.com> writes:

> In this series we deselect a bunch of features when they
> not required, so less objects are built.
>
> While this reduce pressure on CI and slow systems, this is
> particularly helpful for developers regularly testing multiple
> build configurations.
>
> All CI tests pass:
> https://gitlab.com/philmd/qemu/-/pipelines/245654160
>
> Supersedes: <20210120151916.1167448-1-philmd@redhat.com>

Series looks good to me but I guess you need some sub-system feedback.
I've sent a few more patches that might be worth rolling in to better
handle check-tcg/softfloat.

>
> Philippe Mathieu-Daudé (12):
>   configure: Only check for audio drivers if system-mode is selected
>   tests/meson: Only build softfloat objects if TCG is selected
>   pc-bios/meson: Only install EDK2 blob firmwares with system emulation
>   meson: Do not build optional libraries by default
>   meson: Restrict block subsystem processing
>   meson: Merge trace_events_subdirs array
>   meson: Restrict some trace event directories to user/system emulation
>   meson: Restrict emulation code
>   qapi/meson: Restrict qdev code to system-mode emulation
>   qapi/meson: Remove QMP from user-mode emulation
>   qapi/meson: Restrict system-mode specific modules
>   qapi/meson: Restrict UI module to system emulation and tools
>
>  configure           |  6 +++++
>  meson.build         | 55 ++++++++++++++++++++++++++-------------------
>  stubs/qdev.c        | 23 +++++++++++++++++++
>  MAINTAINERS         |  1 +
>  pc-bios/meson.build |  1 +
>  qapi/meson.build    | 34 +++++++++++++++++++---------
>  stubs/meson.build   |  2 ++
>  tests/meson.build   | 11 +++++++--
>  8 files changed, 97 insertions(+), 36 deletions(-)
>  create mode 100644 stubs/qdev.c


-- 
Alex Bennée