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

On Mon, Oct 5, 2020 at 2:40 PM Andrea Bolognani <abologna(a)redhat.com&gt; 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.