Rewrite local headers check to be more robust and informative.

This commit is contained in:
Roland McGrath 2011-07-02 15:52:51 -07:00
parent 1b74661a6b
commit ea5ee9f73d
3 changed files with 23 additions and 15 deletions

View File

@ -1,5 +1,9 @@
2011-07-02 Roland McGrath <roland@hack.frob.com>
* scripts/check-local-headers.sh: Rewritten using awk.
Match by word, not by line. Print error messages for matches.
* Makefile ($(objpfx)check-local-headers.out): Pass AWK in to it.
* Makerules [shlib-lds-flags empty]:
($(common-objpfx)libc_pic.opts): New target.
($(common-objpfx)libc_pic.os.clean): New target.

View File

@ -284,7 +284,8 @@ endif
endif
$(objpfx)check-local-headers.out: scripts/check-local-headers.sh
scripts/check-local-headers.sh "$(includedir)" "$(objpfx)" > $@
AWK='$(AWK)' scripts/check-local-headers.sh \
"$(includedir)" "$(objpfx)" > $@
ifneq ($(PERL),no)
installed-headers = argp/argp.h assert/assert.h catgets/nl_types.h \

View File

@ -1,5 +1,5 @@
#! /bin/bash
# Copyright (C) 2005, 2007, 2009 Free Software Foundation, Inc.
# Copyright (C) 2005,2007,2009,2011 Free Software Foundation, Inc.
# This file is part of the GNU C Library.
# The GNU C Library is free software; you can redistribute it and/or
@ -23,18 +23,21 @@ objpfx="$2"
# To avoid long paths.
cd "$objpfx"
# Search all dependency files for file names in the include directory.
# There are a few system headers we are known to use.
if fgrep "$includedir" */*.{o,os,oS}.d |
fgrep -v "$includedir/asm" |
fgrep -v "$includedir/linux" |
fgrep -v "$includedir/selinux" |
fgrep -v "$includedir/sys/capability.h" |
fgrep -v "$includedir/gd" |
fgrep -v "$includedir/nss3"; then
# If we found a match something is wrong.
exit 1
fi
exit 0
exec ${AWK} -v includedir="$includedir" '
BEGIN {
status = 0
exclude = "^" includedir \
"/(asm[-/]|linux/|selinux/|gd|nss3/|sys/capability\\.h)"
}
/^[^ ]/ && $1 ~ /.*:/ { obj = $1 }
{
for (i = 1; i <= NF; ++i) {
if ($i ~ ("^" includedir) && $i !~ exclude) {
print "***", obj, "uses", $i
status = 1
}
}
}
END { exit status }' */*.{o,os,oS}.d