From 527b3c1da57affda6bf1c5991d441490e8932c0c Mon Sep 17 00:00:00 2001 From: Martin Roth Date: Thu, 9 Feb 2017 16:44:24 -0800 Subject: [PATCH] UPSTREAM: util/lint: Don't check license text for files with under 5 lines BUG=none BRANCH=none TEST=none Change-Id: Ie3271826bc6a5094d3a376a0a44a7be5e3c696c9 Signed-off-by: Furquan Shaikh Original-Commit-Id: 03e9d6aa13f474ab980779366b7a6c8593721be6 Original-Change-Id: I7c1e3cf558d447838819b4d6a63d93d48d5f13e0 Original-Signed-off-by: Martin Roth Original-Reviewed-on: https://review.coreboot.org/18316 Original-Tested-by: build bot (Jenkins) Original-Reviewed-by: Jonathan Neuschfer Original-Reviewed-by: Werner Zeh Reviewed-on: https://chromium-review.googlesource.com/443679 --- util/lint/lint-000-license-headers | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/util/lint/lint-000-license-headers b/util/lint/lint-000-license-headers index 0d2a3af5e8..e705929ce5 100755 --- a/util/lint/lint-000-license-headers +++ b/util/lint/lint-000-license-headers @@ -93,8 +93,10 @@ check_for_license '"AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES' check_for_license 'No license required' for file in $headerlist; do - #verify the file exists, and has content that requires a header - if [ -f "$file" ] && [ "$(wc -l < "$file")" -ne 0 ]; then + # Verify the file exists, and has content that requires a header + # This assumes that a file that has 4 lines or fewer is not notable + # enough to require a license. + if [ -f "$file" ] && [ "$(wc -l < "$file")" -gt 4 ]; then echo "$file has no recognized license header." fi done