[libvirt PATCH 10/13] selinux: remove duplicate sources list for policy

Daniel P. Berrangé berrange at redhat.com
Fri Aug 6 17:48:07 UTC 2021


By using the 'files()' function, the sources retain knowledge of the
path in which they are held. This avoids the need to duplicate the
source list in each subdir.

Signed-off-by: Daniel P. Berrangé <berrange at redhat.com>
---
 src/security/selinux/mcs/meson.build | 6 ------
 src/security/selinux/meson.build     | 6 ++++++
 src/security/selinux/mls/meson.build | 6 ------
 3 files changed, 6 insertions(+), 12 deletions(-)

diff --git a/src/security/selinux/mcs/meson.build b/src/security/selinux/mcs/meson.build
index 9ecfe976db..4ef9f0b427 100644
--- a/src/security/selinux/mcs/meson.build
+++ b/src/security/selinux/mcs/meson.build
@@ -1,9 +1,3 @@
-selinux_sources = [
-  '../virt.te',
-  '../virt.if',
-  '../virt.fc',
-]
-
 # targeted/minimum policy module
 virt_pp = custom_target('virt.pp',
   output : 'virt.pp',
diff --git a/src/security/selinux/meson.build b/src/security/selinux/meson.build
index af5a5e38cb..6f84c4e067 100644
--- a/src/security/selinux/meson.build
+++ b/src/security/selinux/meson.build
@@ -33,6 +33,12 @@ if not selinux_policy_opt.disabled()
     install_data('virt.if',
                  install_dir : 'share/selinux/devel/include/distributed')
 
+    selinux_sources = files([
+      'virt.te',
+      'virt.if',
+      'virt.fc',
+    ])
+
     subdir('mcs')
     subdir('mls')
   endif
diff --git a/src/security/selinux/mls/meson.build b/src/security/selinux/mls/meson.build
index ef72a5f5ec..4d0b103ada 100644
--- a/src/security/selinux/mls/meson.build
+++ b/src/security/selinux/mls/meson.build
@@ -1,9 +1,3 @@
-selinux_sources = [
-  '../virt.te',
-  '../virt.if',
-  '../virt.fc',
-]
-
 # MLS policy module
 virt_pp_mls = custom_target('virt.pp',
   output : 'virt.pp',
-- 
2.31.1




More information about the libvir-list mailing list