Re: [libvirt PATCH 1/9] spec: Simplify setting features off by default

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Oct 5, 2020 at 2:40 PM Andrea Bolognani <abologna@xxxxxxxxxx> wrote:
>
> The right-hand side of these expressions will always evaluate to
> zero. Stop obfuscating this fact.
>
> Signed-off-by: Andrea Bolognani <abologna@xxxxxxxxxx>
> ---
>  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!





[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux