Philippe Mathieu-Daudé <philippe.mathieu.da...@gmail.com> writes:
> On 21/3/22 16:30, Alex Bennée wrote:
>> We were not building the vhost-user-blk server due to 32 bit
>> compilation problems. The problem was due to format string types so
>> fix that and then enable the build. Tweak the rule to follow the same
>> rules as other vhost-user daemons.
>> Signed-off-by: Alex Bennée <alex.ben...@linaro.org>
>> ---
>> meson.build | 2 +-
>> contrib/vhost-user-blk/vhost-user-blk.c | 6 +++---
>> contrib/vhost-user-blk/meson.build | 3 +--
>> 3 files changed, 5 insertions(+), 6 deletions(-)
>> diff --git a/meson.build b/meson.build
>> index 282e7c4650..0435419307 100644
>> --- a/meson.build
>> +++ b/meson.build
>> @@ -1326,7 +1326,7 @@ have_vhost_user_blk_server =
>> get_option('vhost_user_blk_server') \
>> error_message: 'vhost_user_blk_server requires linux') \
>> .require('CONFIG_VHOST_USER' in config_host,
>> error_message: 'vhost_user_blk_server requires vhost-user
>> support') \
>> - .disable_auto_if(not have_system) \
>> + .disable_auto_if(not have_tools and not have_system) \
>
> s/and/or/?
AIUI this is for auto-enabling so you probably want to build if you have
system build or are explicitly building tools. I think, it's confusing
being in a double negative.
>
>> .allowed()
--
Alex Bennée
