Like libvirt.h, these two public headers also shouldn't be included
in internal source, except "internal.h". A later patch will force
tools to use <> form.
---
 cfg.mk             | 20 ++++++++++++++++++++
 daemon/remote.c    |  2 --
 src/libvirt-qemu.c |  1 -
 3 files changed, 20 insertions(+), 3 deletions(-)

diff --git a/cfg.mk b/cfg.mk
index 98c7e40..4b84bde 100644
--- a/cfg.mk
+++ b/cfg.mk
@@ -758,6 +758,20 @@ sc_prohibit_include_virterror_h:
        halt='Do not include libvirt/virterror.h in internal source'    \
          $(_sc_search_regexp)
 
+# Don't include "libvirt/libvirt-qemu.h" in "" form.
+sc_prohibit_include_libvirt_qemu_h:
+       @prohibit='^# *include *"libvirt/libvirt-qemu\.h"'                      
\
+       in_vc_files='\.[ch]$$'                                          \
+       halt='Do not include libvirt/libvirt-qemu.h in internal source' \
+         $(_sc_search_regexp)
+
+# Don't include "libvirt/libvirt-lxc.h" in "" form.
+sc_prohibit_include_libvirt_lxc_h:
+       @prohibit='^# *include *"libvirt/libvirt-lxc\.h"'                       
\
+       in_vc_files='\.[ch]$$'                                          \
+       halt='Do not include libvirt/libvirt-lxc.h in internal source'  \
+         $(_sc_search_regexp)
+
 # Don't include "libvirt/libvirt.h" in <> form. Except external tools, e.g.
 # python binding, examples and tools subdirectories.
 sc_prohibit_include_libvirt_h_1:
@@ -939,3 +953,9 @@ exclude_file_name_regexp--sc_prohibit_include_libvirt_h_1 = 
\
 
 exclude_file_name_regexp--sc_prohibit_include_virterror_h_1 = \
   ^(examples/|tools/|python/|include/libvirt/)
+
+exclude_file_name_regexp--sc_prohibit_include_libvirt_qemu_h = \
+  ^(src/internal\.h$$|tools/)
+
+exclude_file_name_regexp--sc_prohibit_include_libvirt_lxc_h = \
+  ^(src/internal\.h$$|tools/)
diff --git a/daemon/remote.c b/daemon/remote.c
index 45c50f3..c559d6f 100644
--- a/daemon/remote.c
+++ b/daemon/remote.c
@@ -38,8 +38,6 @@
 #include "virutil.h"
 #include "stream.h"
 #include "viruuid.h"
-#include "libvirt/libvirt-qemu.h"
-#include "libvirt/libvirt-lxc.h"
 #include "vircommand.h"
 #include "intprops.h"
 #include "virnetserverservice.h"
diff --git a/src/libvirt-qemu.c b/src/libvirt-qemu.c
index 11da2f3..fb19584 100644
--- a/src/libvirt-qemu.c
+++ b/src/libvirt-qemu.c
@@ -26,7 +26,6 @@
 #include "virerror.h"
 #include "virlog.h"
 #include "datatypes.h"
-#include "libvirt/libvirt-qemu.h"
 
 #define VIR_FROM_THIS VIR_FROM_NONE
 
-- 
1.8.1.4

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to