[libvirt] [PATCH 15/27] libvirt.spec.in: remove polkit conditional

Daniel P. Berrange berrange at redhat.com
Wed May 4 16:17:22 UTC 2016


Both RHEL and Fedora build with polkit enabled at all
times.

Signed-off-by: Daniel P. Berrange <berrange at redhat.com>
---
 libvirt.spec.in | 26 +++++---------------------
 1 file changed, 5 insertions(+), 21 deletions(-)

diff --git a/libvirt.spec.in b/libvirt.spec.in
index f2b83b6..01c7b6a 100644
--- a/libvirt.spec.in
+++ b/libvirt.spec.in
@@ -79,7 +79,6 @@
 %define with_selinux          0%{!?_without_selinux:1}
 
 # A few optional bits off by default, we enable later
-%define with_polkit        0%{!?_without_polkit:0}
 %define with_fuse          0%{!?_without_fuse:0}
 %define with_yajl          0%{!?_without_yajl:0}
 %define with_nwfilter      0%{!?_without_nwfilter:0}
@@ -170,7 +169,6 @@
     %define with_fuse      0%{!?_without_fuse:1}
 %endif
 
-%define with_polkit    0%{!?_without_polkit:1}
 %define with_yajl      0%{!?_without_yajl:1}
 
 # Enable sanlock library for lock management with QEMU
@@ -378,14 +376,12 @@ BuildRequires: ebtables
 %endif
 BuildRequires: module-init-tools
 BuildRequires: cyrus-sasl-devel
-%if %{with_polkit}
-    %if 0%{?fedora} || 0%{?rhel} >= 7
+%if 0%{?fedora} || 0%{?rhel} >= 7
 # F22 polkit-devel doesn't pull in polkit anymore, which we need for pkcheck
 BuildRequires: polkit >= 0.112
 BuildRequires: polkit-devel >= 0.112
-    %else
+%else
 BuildRequires: polkit-devel >= 0.93
-    %endif
 %endif
 %if %{with_storage_fs}
 # For mount/umount in FS driver
@@ -462,10 +458,8 @@ BuildRequires: util-linux
 BuildRequires: nfs-utils
 %endif
 
-%if %{with_firewalld} || %{with_polkit}
 # Communication with the firewall and polkit daemons use DBus
 BuildRequires: dbus-devel
-%endif
 
 # Fedora build root suckage
 BuildRequires: gawk
@@ -511,12 +505,10 @@ Requires: module-init-tools
 # for /sbin/ip & /sbin/tc
 Requires: iproute
 Requires: avahi-libs
-%if %{with_polkit}
-    %if 0%{?fedora} || 0%{?rhel} >= 7
+%if 0%{?fedora} || 0%{?rhel} >= 7
 Requires: polkit >= 0.112
-    %else
+%else
 Requires: polkit >= 0.93
-    %endif
 %endif
 %if %{with_cgconfig}
 Requires: libcgroup
@@ -1088,10 +1080,6 @@ rm -rf .git
     %define _without_vmware --without-vmware
 %endif
 
-%if ! %{with_polkit}
-    %define _without_polkit --without-polkit
-%endif
-
 %if ! %{with_uml}
     %define _without_uml --without-uml
 %endif
@@ -1223,7 +1211,7 @@ rm -f po/stamp-po
            %{?_without_libxl} \
            --with-sasl \
            --with-avahi \
-           %{?_without_polkit} \
+           --with-polkit \
            --with-libvirtd \
            %{?_without_uml} \
            %{?_without_phyp} \
@@ -1385,12 +1373,10 @@ then
 fi
 
 %pre daemon
-%if %{with_polkit}
 # 'libvirt' group is just to allow password-less polkit access to
 # libvirtd. The uid number is irrelevant, so we use dynamic allocation
 # described at the above link.
 getent group libvirt >/dev/null || groupadd -r libvirt
-%endif
 
 exit 0
 
@@ -1708,11 +1694,9 @@ exit 0
 %{_datadir}/augeas/lenses/tests/test_libvirt_lockd.aug
 %endif
 
-%if %{with_polkit}
 %{_datadir}/polkit-1/actions/org.libvirt.unix.policy
 %{_datadir}/polkit-1/actions/org.libvirt.api.policy
 %{_datadir}/polkit-1/rules.d/50-libvirt.rules
-%endif
 
 %dir %attr(0700, root, root) %{_localstatedir}/log/libvirt/
 
-- 
2.5.5




More information about the libvir-list mailing list