Skip to content

Commit

Permalink
Makefile.am: Avoid include/ depending on lib/.
Browse files Browse the repository at this point in the history
Add a build check that public openvswitch header file should not include
internal library.

Suggested-by: Joe Stringer <joe@ovn.org>
Suggested-by: Daniele Di Proietto <diproiettod@vmware.com>
Signed-off-by: Yi-Hung Wei <yihung.wei@gmail.com>
Signed-off-by: Joe Stringer <joe@ovn.org>
  • Loading branch information
YiHungWei authored and joestringer committed Feb 21, 2017
1 parent aafee63 commit c458e4a
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions Makefile.am
Expand Up @@ -230,6 +230,13 @@ config-h-check:
echo "every C source file must #include <config.h>."; \
exit 1; \
fi
if grep '#include' $(srcdir)/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.
Expand Down

0 comments on commit c458e4a

Please sign in to comment.