[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] meson: always include contrib/libvhost-user
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH] meson: always include contrib/libvhost-user |
Date: |
Sun, 8 Nov 2020 16:54:30 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.3.1 |
On 11/6/20 10:03 PM, Stefan Hajnoczi wrote:
> libvhost-user is needed when CONFIG_LINUX is set. The CONFIG_VHOST_USER
> check in meson.build is incorrect.
>
> In fact, no explicit check is needed since this dependency is not built
> by default. If something declares a dependency on libvhost-user then it
> will be built, otherwise it won't be built (i.e. on non-Linux hosts).
>
> This fixes ./configure --disable-vhost-user && make.
>
> Fixes: bc15e44cb2191bbb2318878acdf5038134e56394 ("configure: introduce
> --enable-vhost-user-blk-server")
> Reported-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> Reported-by: Michael S. Tsirkin <mst@redhat.com>
> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
> ---
> meson.build | 6 +-----
> 1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/meson.build b/meson.build
> index f5175010df..b473620321 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -1450,11 +1450,7 @@ trace_events_subdirs += [
> 'util',
> ]
>
> -vhost_user = not_found
> -if 'CONFIG_VHOST_USER' in config_host
> - subdir('contrib/libvhost-user')
> -endif
> -
> +subdir('contrib/libvhost-user')
I haven't reviewed what this code does, but testing it I see:
$ ../configure --disable-vhost-user
vhost-user support: YES
which doesn't look correct...
> subdir('qapi')
> subdir('qobject')
> subdir('stubs')
>