> -----Original Message-----
> From: Qemu-devel <qemu-devel-
> bounces+chen.zhang=intel....@nongnu.org> On Behalf Of Thomas Huth
> Sent: Wednesday, June 22, 2022 10:03 PM
> To: qemu-devel@nongnu.org; Riku Voipio <riku.voi...@iki.fi>
> Cc: Michael Tokarev <m...@tls.msk.ru>; qemu-triv...@nongnu.org; Paolo
> Bonzini <pbonz...@redhat.com>
> Subject: [PATCH] common-user: Only compile the common user code if
> have_user is set
>
> There is no need to waste cycles here if we only compile the system binaries
> or tools. Additionally, this change is even a hard requirement for building
> the
> tools on systems that do not have an entry in the common-user/host/ folder
> (since common-user/meson.build is trying to add such a path via the
> include_directories() command).
>
> Reported-by: Michael Tokarev <m...@tls.msk.ru>
> Signed-off-by: Thomas Huth <th...@redhat.com>
Looks good to me.
Reviewed-by: Zhang Chen <chen.zh...@intel.com>
> ---
> common-user/meson.build | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/common-user/meson.build b/common-user/meson.build index
> 26212dda5c..ac9de5b9e3 100644
> --- a/common-user/meson.build
> +++ b/common-user/meson.build
> @@ -1,3 +1,7 @@
> +if not have_user
> + subdir_done()
> +endif
> +
> common_user_inc += include_directories('host/' / host_arch)
>
> user_ss.add(files(
> --
> 2.31.1
>
