[libvirt PATCH 1/9] spec: Simplify setting features off by default
Neal Gompa
ngompa13 at gmail.com
Tue Oct 6 00:42:57 UTC 2020
On Mon, Oct 5, 2020 at 2:40 PM Andrea Bolognani <abologna at redhat.com> wrote:
>
> The right-hand side of these expressions will always evaluate to
> zero. Stop obfuscating this fact.
>
> Signed-off-by: Andrea Bolognani <abologna at redhat.com>
> ---
> libvirt.spec.in | 18 +++++++++---------
> 1 file changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/libvirt.spec.in b/libvirt.spec.in
> index d13aae5cf5..815ab246e9 100644
> --- a/libvirt.spec.in
> +++ b/libvirt.spec.in
> @@ -85,15 +85,15 @@
> %endif
>
> # A few optional bits off by default, we enable later
> -%define with_fuse 0%{!?_without_fuse:0}
> -%define with_sanlock 0%{!?_without_sanlock:0}
> -%define with_numad 0%{!?_without_numad:0}
> -%define with_firewalld 0%{!?_without_firewalld:0}
> -%define with_firewalld_zone 0%{!?_without_firewalld_zone:0}
> -%define with_libssh2 0%{!?_without_libssh2:0}
> -%define with_wireshark 0%{!?_without_wireshark:0}
> -%define with_libssh 0%{!?_without_libssh:0}
> -%define with_bash_completion 0%{!?_without_bash_completion:0}
> +%define with_fuse 0
> +%define with_sanlock 0
> +%define with_numad 0
> +%define with_firewalld 0
> +%define with_firewalld_zone 0
> +%define with_libssh2 0
> +%define with_wireshark 0
> +%define with_libssh 0
> +%define with_bash_completion 0
>
> # Finally set the OS / architecture specific special cases
>
This crazy setup is an attempt to implement rpm %bcond behavior
without actually using %bcond statements. Instead of removing the
right-hand-side conditionals, perhaps it'd be better to just finally
make the switch to %bcond statements.
--
真実はいつも一つ!/ Always, there's only one truth!
More information about the libvir-list
mailing list