[coreboot-gerrit] New patch to review for coreboot: util/lint: Get ready to add util/iotools

Martin Roth (martinroth@google.com) gerrit at coreboot.org
Wed Oct 12 22:41:02 CEST 2016


Martin Roth (martinroth at google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17008

-gerrit

commit 21cdd61f9d4f5c8cfcc6e9ef786b18f909e62aa9
Author: Martin Roth <martinroth at google.com>
Date:   Wed Oct 12 12:04:29 2016 -0600

    util/lint: Get ready to add util/iotools
    
    Google's iotools package breaks a couple of the coreboot lint rules.
    To add it exactly as it is in the package, I'm disabling those lint
    rules for the util/iotools directory.  After the package is added,
    I'll update it in a follow-on commit, and re-enable the lint packages.
    
    Change-Id: Ibcfdf031349ed1aee6948e23e9488452b3acf581
    Signed-off-by: Martin Roth <martinroth at google.com>
---
 util/lint/lint-stable-003-whitespace   | 2 +-
 util/lint/lint-stable-009-old-licenses | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/util/lint/lint-stable-003-whitespace b/util/lint/lint-stable-003-whitespace
index 3dbd473..1b7ef78 100755
--- a/util/lint/lint-stable-003-whitespace
+++ b/util/lint/lint-stable-003-whitespace
@@ -15,5 +15,5 @@
 # DESCR: Check for superfluous whitespace in the tree
 
 LC_ALL=C export LC_ALL
-grep -l "[[:space:]][[:space:]]*$" `git ls-files src util |egrep -v "(^3rdparty|^src/vendorcode/|^util/kconfig/|^util/nvidia/cbootimage$|\<COPYING\>|\<LICENSE\>|\<README\>|_shipped$|\.patch$|\.bin$|\.hex$|\.jpg$)"` | \
+grep -l "[[:space:]][[:space:]]*$" `git ls-files src util |egrep -v "(^util/iotools/|^3rdparty|^src/vendorcode/|^util/kconfig/|^util/nvidia/cbootimage$|\<COPYING\>|\<LICENSE\>|\<README\>|_shipped$|\.patch$|\.bin$|\.hex$|\.jpg$)"` | \
 	sed -e "s,^.*$,File & has lines ending with whitespace.,"
diff --git a/util/lint/lint-stable-009-old-licenses b/util/lint/lint-stable-009-old-licenses
index fdc76aa..4094949 100755
--- a/util/lint/lint-stable-009-old-licenses
+++ b/util/lint/lint-stable-009-old-licenses
@@ -29,6 +29,7 @@ HEADER_EXCLUDED="\
 \.patch$|\
 _shipped$|\
 ^util/scripts/no-fsf-addresses.sh|\
+^util/iotools/|\
 ^util/lint/lint-000-license-headers|\
 ^util/lint/lint-stable-009-old-licenses\
 "



More information about the coreboot-gerrit mailing list