diff options
author | David Woodhouse <dwmw2@infradead.org> | 2006-06-18 07:02:10 -0400 |
---|---|---|
committer | David Woodhouse <dwmw2@infradead.org> | 2006-06-18 07:02:10 -0400 |
commit | 684753599afc76aa8f66c731bafb7204b39265b8 (patch) | |
tree | 660da6e957637f063735c5f27090674d4249c573 | |
parent | 8d730cfb50cc77da6d00f941daef440918a1922f (diff) |
Basic implementation of 'make headers_check'
Based on the 'headers_install' target, this performs a basic sanity check
on the exported headers -- so far only checking that they do not include
any other headers which aren't selected for import, but easily extendable.
Signed-off-by: David Woodhouse <dwmw2@infradead.org>
-rw-r--r-- | Makefile | 4 | ||||
-rw-r--r-- | scripts/Makefile.headersinst | 11 | ||||
-rwxr-xr-x | scripts/hdrcheck.sh | 8 |
3 files changed, 23 insertions, 0 deletions
@@ -866,6 +866,10 @@ headers_install: include/linux/version.h | |||
866 | $(Q)rm -rf $(INSTALL_HDR_PATH)/include | 866 | $(Q)rm -rf $(INSTALL_HDR_PATH)/include |
867 | $(Q)$(MAKE) -rR -f $(srctree)/scripts/Makefile.headersinst obj=include | 867 | $(Q)$(MAKE) -rR -f $(srctree)/scripts/Makefile.headersinst obj=include |
868 | 868 | ||
869 | PHONY += headers_check | ||
870 | headers_check: headers_install | ||
871 | $(Q)$(MAKE) -rR -f $(srctree)/scripts/Makefile.headersinst obj=include HDRCHECK=1 | ||
872 | |||
869 | # --------------------------------------------------------------------------- | 873 | # --------------------------------------------------------------------------- |
870 | # Modules | 874 | # Modules |
871 | 875 | ||
diff --git a/scripts/Makefile.headersinst b/scripts/Makefile.headersinst index 688f8cb081d9..aa9990a3ccd6 100644 --- a/scripts/Makefile.headersinst +++ b/scripts/Makefile.headersinst | |||
@@ -78,6 +78,11 @@ quiet_cmd_unifdef = UNIFDEF $(_dst)/$@ | |||
78 | cmd_unifdef = $(UNIFDEF) $(srctree)/$(obj)/$@ | $(HDRSED) \ | 78 | cmd_unifdef = $(UNIFDEF) $(srctree)/$(obj)/$@ | $(HDRSED) \ |
79 | > $(INSTALL_HDR_PATH)/$(_dst)/$@ || : | 79 | > $(INSTALL_HDR_PATH)/$(_dst)/$@ || : |
80 | 80 | ||
81 | quiet_cmd_check = CHECK $(_dst)/$@ | ||
82 | cmd_check = $(srctree)/scripts/hdrcheck.sh \ | ||
83 | $(INSTALL_HDR_PATH)/include \ | ||
84 | $(INSTALL_HDR_PATH)/$(_dst)/$@ | ||
85 | |||
81 | quiet_cmd_mkdir = MKDIR $@ | 86 | quiet_cmd_mkdir = MKDIR $@ |
82 | cmd_mkdir = mkdir -p $(INSTALL_HDR_PATH)/$@ | 87 | cmd_mkdir = mkdir -p $(INSTALL_HDR_PATH)/$@ |
83 | 88 | ||
@@ -112,6 +117,11 @@ __headersinst: $(subdir-y) $(header-y) $(unifdef-y) $(altarch-y) $(objhdr-y) | |||
112 | 117 | ||
113 | .PHONY: $(header-y) $(unifdef-y) $(subdir-y) | 118 | .PHONY: $(header-y) $(unifdef-y) $(subdir-y) |
114 | 119 | ||
120 | ifdef HDRCHECK | ||
121 | # Rules for checking headers | ||
122 | $(objhdr-y) $(header-y) $(unifdef-y): | ||
123 | $(call cmd,check) | ||
124 | else | ||
115 | # Rules for installing headers | 125 | # Rules for installing headers |
116 | 126 | ||
117 | $(objhdr-y) $(subdir-y) $(header-y) $(unifdef-y): $(_dst) | 127 | $(objhdr-y) $(subdir-y) $(header-y) $(unifdef-y): $(_dst) |
@@ -134,6 +144,7 @@ $(header-y): | |||
134 | $(unifdef-y): | 144 | $(unifdef-y): |
135 | $(call cmd,unifdef) | 145 | $(call cmd,unifdef) |
136 | endif | 146 | endif |
147 | endif | ||
137 | 148 | ||
138 | hdrinst := -rR -f $(srctree)/scripts/Makefile.headersinst obj | 149 | hdrinst := -rR -f $(srctree)/scripts/Makefile.headersinst obj |
139 | 150 | ||
diff --git a/scripts/hdrcheck.sh b/scripts/hdrcheck.sh new file mode 100755 index 000000000000..b3bb683b56b6 --- /dev/null +++ b/scripts/hdrcheck.sh | |||
@@ -0,0 +1,8 @@ | |||
1 | #!/bin/sh | ||
2 | |||
3 | for FILE in `grep '^#include <' $2 | cut -f2 -d\< | cut -f1 -d\> | egrep ^linux\|^asm` ; do | ||
4 | if [ ! -r $1/$FILE ]; then | ||
5 | echo $2 requires $FILE, which does not exist | ||
6 | exit 1 | ||
7 | fi | ||
8 | done | ||