[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH-for-5.2 v2 2/4] meson: Only build vhost-user when system or t
From: |
Stefan Hajnoczi |
Subject: |
Re: [PATCH-for-5.2 v2 2/4] meson: Only build vhost-user when system or tools is enabled |
Date: |
Wed, 11 Nov 2020 15:48:46 +0000 |
On Wed, Nov 11, 2020 at 01:09:10PM +0100, Philippe Mathieu-Daudé wrote:
> It does not make sense to select vhost-user features
> without system-mode or tools. Return an error when
> this configuration is selected. Example:
>
> $ ../configure --disable-tools --disable-system
> --enable-vhost-user-blk-server
>
> ../meson.build:755:4: ERROR: Problem encountered: vhost-user does not make
> sense without system or tools support enabled
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
> meson.build | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/meson.build b/meson.build
> index 4b789f18c17..4fc58eb2c3d 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -751,6 +751,10 @@
>
> has_statx = cc.links(statx_test)
>
> +if 'CONFIG_VHOST_USER' in config_host and not (have_system or have_tools)
> + error('vhost-user does not make sense without system or tools support
> enabled')
> +endif
Now the following fails on Linux hosts:
$ ./configure --disable-tools --disable-system
../meson.build:755:4: ERROR: Problem encountered: vhost-user does not make
sense without system or tools support enabled
Previously it would succeed and make would build qemu-user binaries,
documentation, trace-events-all, etc so this looks like a regression.
In addition, adding this error is inconsistent with all the other
./configure options which do not check whether the build target that
uses them has been disabled. We'd need to implement the same check for
every option to make ./configure consistent. For example, if SPICE is
enabled but --disable-system is given then there should be an error
saying it enabling SPICE does not make sense, etc.
Stefan
signature.asc
Description: PGP signature