[libvirt] [PATCH 5/n RFC] build: avoid stranded file when polkit is not used

Eric Blake eblake at redhat.com
Wed Sep 4 03:07:16 UTC 2013


Untested for now (it's late for me), but seems right...

daemon/Makefile.am installs a .policy file if WITH_POLKIT is set
(which is supposedly the union between WITH_POLKIT0 and WITH_POLKIT1).
src/Makefile.am, on the other hand, installs a .policy file if
WITH_POLKIT1 is set.  When running 'make rpm' with client_only
manually set, on a Fedora 19 box, it appears that WITH_POLKIT1 is
true (the polkit 1 library exists) but WITH_POLKIT is false (the
spec file requests that configure explicitly use --without-polkit),
which leads to this failure:

RPM build errors:
    Installed (but unpackaged) file(s) found:
   /usr/share/polkit-1/actions/org.libvirt.api.policy

Fix it by adding another conditional.

Signed-off-by: Eric Blake <eblake at redhat.com>
---

I'll test this before pushing (and amend the commit message
to drop the first paragraph), even if it gets a positive
review in the meantime.

This patch was much easier to finally figure out once I first
wrote this other patch (still pending review):
https://www.redhat.com/archives/libvir-list/2013-September/msg00195.html

 src/Makefile.am | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/Makefile.am b/src/Makefile.am
index 9b336fd..37beb6c 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -1497,7 +1497,9 @@ if WITH_POLKIT1
 libvirt_driver_access_la_SOURCES += $(ACCESS_DRIVER_POLKIT_SOURCES)

 polkitactiondir = $(datadir)/polkit-1/actions
+if WITH_POLKIT
 polkitaction_DATA = $(ACCESS_DRIVER_POLKIT_POLICY)
+endif WITH_POLKIT

 $(ACCESS_DRIVER_POLKIT_POLICY): $(srcdir)/access/viraccessperm.h \
     $(srcdir)/access/genpolkit.pl Makefile.am
-- 
1.8.3.1




More information about the libvir-list mailing list