[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/4] configure: Fix --without-default-features propagation to
From: |
Markus Armbruster |
Subject: |
Re: [PATCH 1/4] configure: Fix --without-default-features propagation to meson |
Date: |
Wed, 14 Jul 2021 07:43:49 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) |
Philippe Mathieu-Daudé <philmd@redhat.com> writes:
> +Alex
>
> On 7/13/21 11:31 AM, Thomas Huth wrote:
>> A typo prevents that many features get disabled when the user
>> runs "configure" with the --without-default-features switch.
>>
>> Reported-by: Cole Robinson <crobinso@redhat.com>
>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>> ---
>> configure | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/configure b/configure
>> index 85db248ac1..229ea52516 100755
>> --- a/configure
>> +++ b/configure
>> @@ -5205,7 +5205,7 @@ if test "$skip_meson" = no; then
>> -Ddocs=$docs -Dsphinx_build=$sphinx_build -Dinstall_blobs=$blobs \
>> -Dvhost_user_blk_server=$vhost_user_blk_server
>> -Dmultiprocess=$multiprocess \
>> -Dfuse=$fuse -Dfuse_lseek=$fuse_lseek
>> -Dguest_agent_msi=$guest_agent_msi -Dbpf=$bpf\
>> - $(if test "$default_features" = no; then echo
>> "-Dauto_features=disabled"; fi) \
>> + $(if test "$default_feature" = no; then echo
>> "-Dauto_features=disabled"; fi) \
>
> The option should be name plural (default_features)...
Actually, no. The variable holds the initial value for the feature
variables that are to be controlled by --with-default-features.
Example:
vde="$default_feature"
Perhaps $feature_default would be a better name.
> What is 'auto_features' used for?
>
>> -Dtcg_interpreter=$tcg_interpreter \
>> $cross_arg \
>> "$PWD" "$source_path"
>>
- [PATCH 0/4] Fixes for the --without-default-features configure switch, Thomas Huth, 2021/07/13
- [PATCH 2/4] configure: Allow vnc to get disabled with --without-default-features, Thomas Huth, 2021/07/13
- [PATCH 3/4] configure: Fix the default setting of the "xen" feature, Thomas Huth, 2021/07/13
- [PATCH 4/4] configure: Let --without-default-features disable vhost-kernel and vhost-vdpa, Thomas Huth, 2021/07/13
- Re: [PATCH 0/4] Fixes for the --without-default-features configure switch, Cole Robinson, 2021/07/13
- Re: [PATCH 0/4] Fixes for the --without-default-features configure switch, Paolo Bonzini, 2021/07/14