[libvirt] [PATCH 10/17] make: split security driver build rules into security/Makefile.inc.am

Daniel P. Berrangé berrange at redhat.com
Wed Feb 28 10:14:28 UTC 2018


Signed-off-by: Daniel P. Berrangé <berrange at redhat.com>
---
 src/Makefile.am              | 65 +----------------------------------
 src/security/Makefile.inc.am | 80 ++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 81 insertions(+), 64 deletions(-)
 create mode 100644 src/security/Makefile.inc.am

diff --git a/src/Makefile.am b/src/Makefile.am
index 8f628d2d38..7dd50dacec 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -93,6 +93,7 @@ man8_MANS =
 
 include conf/Makefile.inc.am
 include cpu/Makefile.inc.am
+include security/Makefile.inc.am
 include uml/Makefile.inc.am
 include phyp/Makefile.inc.am
 include test/Makefile.inc.am
@@ -636,30 +637,12 @@ check-local: check-protocol check-symfile check-symsorting \
 .PHONY: check-protocol $(PROTOCOL_STRUCTS:structs=struct)
 
 
-SECURITY_DRIVER_APPARMOR_HELPER_SOURCES = \
-		$(DATATYPES_SOURCES) \
-		security/virt-aa-helper.c
-
 
 
 UTIL_IO_HELPER_SOURCES = \
 		util/iohelper.c
 
 
-# Security framework and drivers for various models
-SECURITY_DRIVER_SOURCES = \
-		security/security_driver.h security/security_driver.c \
-		security/security_nop.h security/security_nop.c \
-		security/security_stack.h security/security_stack.c \
-		security/security_dac.h security/security_dac.c \
-		security/security_manager.h security/security_manager.c
-
-SECURITY_DRIVER_SELINUX_SOURCES = \
-		security/security_selinux.h security/security_selinux.c
-
-SECURITY_DRIVER_APPARMOR_SOURCES = \
-		security/security_apparmor.h security/security_apparmor.c
-
 ACCESS_DRIVER_GENERATED = \
 		access/viraccessapicheck.h \
 		access/viraccessapicheck.c \
@@ -768,22 +751,6 @@ GENERATED_SYM_FILES += \
 
 
 
-libvirt_security_manager_la_SOURCES = $(SECURITY_DRIVER_SOURCES)
-noinst_LTLIBRARIES += libvirt_security_manager.la
-libvirt_la_BUILT_LIBADD += libvirt_security_manager.la
-libvirt_security_manager_la_CFLAGS = \
-		-I$(srcdir)/conf $(AM_CFLAGS)
-libvirt_security_manager_la_LDFLAGS = $(AM_LDFLAGS)
-libvirt_security_manager_la_LIBADD = $(SECDRIVER_LIBS)
-if WITH_SECDRIVER_SELINUX
-libvirt_security_manager_la_SOURCES += $(SECURITY_DRIVER_SELINUX_SOURCES)
-libvirt_security_manager_la_CFLAGS += $(SELINUX_CFLAGS)
-endif WITH_SECDRIVER_SELINUX
-if WITH_SECDRIVER_APPARMOR
-libvirt_security_manager_la_SOURCES += $(SECURITY_DRIVER_APPARMOR_SOURCES)
-libvirt_security_manager_la_CFLAGS += $(APPARMOR_CFLAGS)
-endif WITH_SECDRIVER_APPARMOR
-
 libvirt_driver_access_la_SOURCES = \
 	$(ACCESS_DRIVER_SOURCES) $(ACCESS_DRIVER_GENERATED)
 noinst_LTLIBRARIES += libvirt_driver_access.la
@@ -878,8 +845,6 @@ access/viraccessapichecklxc.c: $(srcdir)/rpc/gendispatch.pl \
 
 # Add all conditional sources just in case...
 EXTRA_DIST += \
-		$(SECURITY_DRIVER_SELINUX_SOURCES) \
-		$(SECURITY_DRIVER_APPARMOR_SOURCES) \
 		$(ACCESS_DRIVER_POLKIT_POLICY)
 
 check-local: check-augeas
@@ -1872,34 +1837,6 @@ libvirt_sanlock_helper_LDADD = libvirt.la
 endif WITH_SANLOCK
 
 
-if WITH_SECDRIVER_APPARMOR
-if WITH_LIBVIRTD
-libexec_PROGRAMS += virt-aa-helper
-
-virt_aa_helper_SOURCES = $(SECURITY_DRIVER_APPARMOR_HELPER_SOURCES)
-
-virt_aa_helper_LDFLAGS = \
-		$(AM_LDFLAGS) \
-		$(PIE_LDFLAGS) \
-		$(NULL)
-virt_aa_helper_LDADD = \
-		libvirt.la \
-		libvirt_conf.la \
-		libvirt_util.la \
-		libvirt_driver_storage_impl.la \
-		../gnulib/lib/libgnu.la
-if WITH_DTRACE_PROBES
-virt_aa_helper_LDADD += libvirt_probes.lo
-endif WITH_DTRACE_PROBES
-virt_aa_helper_CFLAGS = \
-		-I$(srcdir)/conf \
-		-I$(srcdir)/security \
-		$(AM_CFLAGS) \
-		$(PIE_CFLAGS) \
-		$(NULL)
-endif WITH_LIBVIRTD
-endif WITH_SECDRIVER_APPARMOR
-EXTRA_DIST += $(SECURITY_DRIVER_APPARMOR_HELPER_SOURCES)
 
 if WITH_NSS
 noinst_LTLIBRARIES += libvirt-nss.la
diff --git a/src/security/Makefile.inc.am b/src/security/Makefile.inc.am
new file mode 100644
index 0000000000..f88b82df7b
--- /dev/null
+++ b/src/security/Makefile.inc.am
@@ -0,0 +1,80 @@
+SECURITY_DRIVER_APPARMOR_HELPER_SOURCES = \
+	$(DATATYPES_SOURCES) \
+	security/virt-aa-helper.c \
+	$(NULL)
+
+SECURITY_DRIVER_SOURCES = \
+	security/security_driver.h \
+	security/security_driver.c \
+	security/security_nop.h \
+	security/security_nop.c \
+	security/security_stack.h \
+	security/security_stack.c \
+	security/security_dac.h \
+	security/security_dac.c \
+	security/security_manager.h \
+	security/security_manager.c \
+	$(NULL)
+
+SECURITY_DRIVER_SELINUX_SOURCES = \
+	security/security_selinux.h \
+	security/security_selinux.c \
+	$(NULL)
+
+SECURITY_DRIVER_APPARMOR_SOURCES = \
+	security/security_apparmor.h \
+	security/security_apparmor.c \
+	$(NULL)
+
+EXTRA_DIST += \
+	$(SECURITY_DRIVER_SELINUX_SOURCES) \
+	$(SECURITY_DRIVER_APPARMOR_SOURCES) \
+	$(SECURITY_DRIVER_APPARMOR_HELPER_SOURCES) \
+	$(NULL)
+
+libvirt_security_manager_la_SOURCES = $(SECURITY_DRIVER_SOURCES)
+noinst_LTLIBRARIES += libvirt_security_manager.la
+libvirt_la_BUILT_LIBADD += libvirt_security_manager.la
+libvirt_security_manager_la_CFLAGS = \
+	-I$(srcdir)/conf \
+	$(AM_CFLAGS) \
+	$(NULL)
+libvirt_security_manager_la_LDFLAGS = $(AM_LDFLAGS)
+libvirt_security_manager_la_LIBADD =
+
+if WITH_SECDRIVER_SELINUX
+libvirt_security_manager_la_SOURCES += $(SECURITY_DRIVER_SELINUX_SOURCES)
+libvirt_security_manager_la_CFLAGS += $(SELINUX_CFLAGS)
+endif WITH_SECDRIVER_SELINUX
+
+if WITH_SECDRIVER_APPARMOR
+libvirt_security_manager_la_SOURCES += $(SECURITY_DRIVER_APPARMOR_SOURCES)
+libvirt_security_manager_la_CFLAGS += $(APPARMOR_CFLAGS)
+
+if WITH_LIBVIRTD
+libexec_PROGRAMS += virt-aa-helper
+
+virt_aa_helper_SOURCES = $(SECURITY_DRIVER_APPARMOR_HELPER_SOURCES)
+
+virt_aa_helper_LDFLAGS = \
+	$(AM_LDFLAGS) \
+	$(PIE_LDFLAGS) \
+	$(NULL)
+virt_aa_helper_LDADD = \
+	libvirt.la \
+	libvirt_conf.la \
+	libvirt_util.la \
+	libvirt_driver_storage_impl.la \
+	../gnulib/lib/libgnu.la \
+	$(NULL)
+if WITH_DTRACE_PROBES
+virt_aa_helper_LDADD += libvirt_probes.lo
+endif WITH_DTRACE_PROBES
+virt_aa_helper_CFLAGS = \
+	-I$(srcdir)/conf \
+	-I$(srcdir)/security \
+	$(AM_CFLAGS) \
+	$(PIE_CFLAGS) \
+	$(NULL)
+endif WITH_LIBVIRTD
+endif WITH_SECDRIVER_APPARMOR
-- 
2.14.3




More information about the libvir-list mailing list