diff options
| author | Martin Roth <martinroth@google.com> | 2016-08-29 15:40:57 -0600 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2016-09-20 21:43:11 +0200 |
| commit | 1ce2ba6832ad13ea95b5e1184dc696d0a13ce036 (patch) | |
| tree | d4f18130e5d41ead983aee807236ddcbcf30986b | |
| parent | 374c39e3cfbb51927860756d32a77d0afd3752a6 (diff) | |
| download | coreboot-1ce2ba6832ad13ea95b5e1184dc696d0a13ce036.tar.xz | |
lint/lint-007-checkpatch: Update lint script
- Check Kconfig files as well.
- Accept a list of directories to check as a command line argument.
- Only look at src & util directories by default.
- Skip src/vendorcode.
- Remove bypass of payloads/coreinfo/util/kconfig directory, it no
longer exists.
Change-Id: Ia522d3ddc29914220bdaae36ea23ded7338c48fd
Signed-off-by: Martin Roth <martinroth@google.com>
Reviewed-on: https://review.coreboot.org/16359
Tested-by: build bot (Jenkins)
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
| -rwxr-xr-x | util/lint/lint-007-checkpatch | 33 |
1 files changed, 26 insertions, 7 deletions
diff --git a/util/lint/lint-007-checkpatch b/util/lint/lint-007-checkpatch index 9d9d96ebc6..b4033e09c5 100755 --- a/util/lint/lint-007-checkpatch +++ b/util/lint/lint-007-checkpatch @@ -12,12 +12,31 @@ # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the # GNU General Public License for more details. # -# DESCR: Checkpatch on all .c and .h files in the tree +# DESCR: Checkpatch on .c, .h, & Kconfig files in the tree LC_ALL=C export LC_ALL -util/lint/checkpatch.pl --show-types --file $( git ls-files \*.[ch] | \ - grep -v ^payloads/libpayload/util/kconfig | \ - grep -v ^payloads/libpayload/curses/PDCurses-3.4 | \ - grep -v ^payloads/coreinfo/util/kconfig | \ - grep -v ^util/kconfig \ - ) + +# GNU BRE syntax list of files to examine +INCLUDED_FILES='.*\.[ch]\|Kconfig.*$' + +EXCLUDED_DIRS="^payloads/libpayload/util/kconfig\|\ +^payloads/libpayload/curses/PDCurses\|\ +^util/kconfig\|\ +^src/vendorcode" + +# Space separated list of directories to test +if [ "$1" = "" ]; then + INCLUDED_DIRS="src util" +else + INCLUDED_DIRS="$1" +fi + +# We want word splitting here, so disable the shellcheck warnings +# shellcheck disable=SC2046,SC2086 +FILELIST=$( git ls-files $INCLUDED_DIRS | \ + grep $INCLUDED_FILES | \ + grep -v $EXCLUDED_DIRS ) + +for FILE in $FILELIST; do + util/lint/checkpatch.pl --show-types --file --quiet "$FILE" +done |