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

Andrea Bolognani abologna at redhat.com
Tue Oct 6 10:09:12 UTC 2020


On Mon, 2020-10-05 at 20:42 -0400, Neal Gompa wrote:
> On Mon, Oct 5, 2020 at 2:40 PM Andrea Bolognani <abologna at redhat.com> wrote:
> > -%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
> 
> 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.

Do you have any idea what the availability of %bcond is?
Specifically, can we use it on RHEL 7? If so, I can take a stab at
converting our spec file, but I don't think that invalidates the
current series and I would prefer it to be merged first. It will be a
better starting point for the conversion anyway.

-- 
Andrea Bolognani / Red Hat / Virtualization




More information about the libvir-list mailing list