Add a build check that public openvswitch header file should not include
internal library.

Suggested-by: Joe Stringer <[email protected]>
Suggested-by: Daniele Di Proietto <[email protected]>
Signed-off-by: Yi-Hung Wei <[email protected]>
---
 Makefile.am | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/Makefile.am b/Makefile.am
index 4b1b48d..7126ebe 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -230,6 +230,13 @@ config-h-check:
          echo "every C source file must #include <config.h>."; \
          exit 1; \
        fi
+       if grep '#include' include/openvswitch/*.h | \
+           grep -vE '(<.*>)|("openvswitch)|("openflow)'; \
+       then \
+         echo "See above for list of violations of the rule that"; \
+         echo "public openvswitch header file should not include internal 
library."; \
+         exit 1; \
+       fi
 .PHONY: config-h-check
 
 # Check for printf() type modifiers that MSVC doesn't support.
-- 
2.7.4

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to