[coreboot-gerrit] New patch to review for coreboot: util/lint: Fix linter for old license headers

[Martin Roth (martinroth@google.com)]

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

-gerrit

commit fe674c4643048a01e5518bce17a642551e330b9b
Author: Martin Roth <martinroth at google.com>
Date:   Tue Jan 19 21:03:04 2016 -0700

    util/lint: Fix linter for old license headers
    
    Somehow I lost a $ on headerlist between when I tested the script
    and when I submitted it, turning headerlist into plain text instead
    of a variable name.  This makes the test always pass.
    - Fix variable bug.
    - exclude this script from the check.
    - update test for empty HEADER_DIRS variable.
    
    Change-Id: I6080c520bc741e9d689f7c66ee97879afc8ba38c
    Signed-off-by: Martin Roth <martinroth at google.com>
---
 util/lint/lint-stable-009-old-licenses | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/util/lint/lint-stable-009-old-licenses b/util/lint/lint-stable-009-old-licenses
index 702798f..9d9eb9f 100755
--- a/util/lint/lint-stable-009-old-licenses
+++ b/util/lint/lint-stable-009-old-licenses
@@ -29,10 +29,10 @@ HEADER_EXCLUDED="\
 _shipped$|\
 ^util/scripts/no-fsf-addresses.sh|\
 ^util/lint/lint-000-license-headers|\
-^util/lint/lint-stable-000-license-headers\
+^util/lint/lint-stable-009-old-licenses\
 "
 
-if [ "$HEADER_DIRS" = "" ]; then
+if [ -z "$HEADER_DIRS" ]; then
 	HEADER_DIRS="src util"
 fi
 
@@ -42,7 +42,7 @@ headerlist=$(git ls-files $HEADER_DIRS | egrep -v "($HEADER_EXCLUDED)")
 
 #check for the old style header
 headerlist=$(grep -il "You should have received a copy of the GNU" \
-	headerlist 2>/dev/null)
+	$headerlist 2>/dev/null)
 
 for file in $headerlist; do
 	echo "$file has the old GPL header."