Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/43511 )
Change subject: util/lint: Add lint and checkpatch coverage for tests/ dir ......................................................................
util/lint: Add lint and checkpatch coverage for tests/ dir
Signed-off-by: Jan Dabros jsd@semihalf.com Change-Id: I8018b75844e630c9ed46c8bc48f2aa1634bf3369 Reviewed-on: https://review.coreboot.org/c/coreboot/+/43511 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Julius Werner jwerner@chromium.org Reviewed-by: Paul Fagerburg pfagerburg@chromium.org --- M util/lint/lint-007-checkpatch M util/lint/lint-stable-000-license-headers M util/lint/lint-stable-003-whitespace M util/lint/lint-stable-009-old-licenses M util/lint/lint-stable-012-executable-bit 5 files changed, 5 insertions(+), 5 deletions(-)
Approvals: build bot (Jenkins): Verified Julius Werner: Looks good to me, approved Paul Fagerburg: Looks good to me, but someone else must approve
diff --git a/util/lint/lint-007-checkpatch b/util/lint/lint-007-checkpatch index f784541..bc34e6b 100755 --- a/util/lint/lint-007-checkpatch +++ b/util/lint/lint-007-checkpatch @@ -22,7 +22,7 @@
# default: test src and util if [ "$1" = "" ]; then - INCLUDED_DIRS="src util" + INCLUDED_DIRS="src util tests" # special mode: take diff from stdin, but exclude the dirs elif [ "$1" = "diff" ]; then args=$( echo $EXCLUDED_DIRS | \ diff --git a/util/lint/lint-stable-000-license-headers b/util/lint/lint-stable-000-license-headers index a29c5d4..9da4a9a 100755 --- a/util/lint/lint-stable-000-license-headers +++ b/util/lint/lint-stable-000-license-headers @@ -6,4 +6,4 @@ # Top level util/lint/lint-000-license-headers "src/acpi src/arch src/commonlib src/console \ src/cpu src/device src/drivers src/ec src/mainboard src/northbridge \ - src/security src/soc src/southbridge src/superio" + src/security src/soc src/southbridge src/superio tests/" diff --git a/util/lint/lint-stable-003-whitespace b/util/lint/lint-stable-003-whitespace index a6e915d..4304043 100755 --- a/util/lint/lint-stable-003-whitespace +++ b/util/lint/lint-stable-003-whitespace @@ -6,7 +6,7 @@
LC_ALL=C export LC_ALL EXCLUDELIST='^src/vendorcode/|^util/kconfig/|^util/nvidia/cbootimage$|COPYING|LICENSE|README|_shipped$|.patch$|.bin$|.hex$|.jpg$|.ttf$|.woff$|.png$|.eot$|.vbt$|.ico$' -INCLUDELIST="src util payloads Makefile* toolchain.inc" +INCLUDELIST="src util payloads Makefile* toolchain.inc tests"
# shellcheck disable=SC2086,SC2046 if uname | grep -qi "linux"; then diff --git a/util/lint/lint-stable-009-old-licenses b/util/lint/lint-stable-009-old-licenses index 4d0cb4f..f5c1f41 100755 --- a/util/lint/lint-stable-009-old-licenses +++ b/util/lint/lint-stable-009-old-licenses @@ -23,7 +23,7 @@ "
if [ -z "$HEADER_DIRS" ]; then - HEADER_DIRS="src util" + HEADER_DIRS="src util tests" fi
LC_ALL=C export LC_ALL diff --git a/util/lint/lint-stable-012-executable-bit b/util/lint/lint-stable-012-executable-bit index e22dda5..5651a67 100755 --- a/util/lint/lint-stable-012-executable-bit +++ b/util/lint/lint-stable-012-executable-bit @@ -4,5 +4,5 @@ # DESCR: Check that source files are not executable
LC_ALL=C export LC_ALL -git ls-tree --full-tree -r HEAD src |grep "^100[7531][7531][7531] blob " |cut -f2- |grep -v ".sh$" | \ +git ls-tree --full-tree -r HEAD src tests |grep "^100[7531][7531][7531] blob " |cut -f2- |grep -v ".sh$" | \ sed -e "s,^.*$,File & has one or more executable bits set in the file permissions.,"