[coreboot-gerrit] Change in coreboot[master]: lint-stable: Only check files tracked by git (ie source files) for +x

Patrick Georgi (Code Review) gerrit at coreboot.org
Wed Sep 6 13:18:46 CEST 2017


Patrick Georgi has uploaded this change for review. ( https://review.coreboot.org/21419


Change subject: lint-stable: Only check files tracked by git (ie source files) for +x
......................................................................

lint-stable: Only check files tracked by git (ie source files) for +x

Change-Id: I99cbcba7a086ef950f248888a83cf24a4db4aee9
Signed-off-by: Patrick Georgi <pgeorgi at google.com>
---
M util/lint/lint-stable-012-executable-bit
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/19/21419/1

diff --git a/util/lint/lint-stable-012-executable-bit b/util/lint/lint-stable-012-executable-bit
index 5e12c95..e7ff3cb 100755
--- a/util/lint/lint-stable-012-executable-bit
+++ b/util/lint/lint-stable-012-executable-bit
@@ -16,5 +16,5 @@
 # DESCR: Check that source files are not executable
 
 LC_ALL=C export LC_ALL
-find src -type f -a \( -perm -100 -o -perm -010 -o -perm -001 \)  -a \! -name '*.sh'| \
+git ls-tree --full-tree -r HEAD src |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.,"

-- 
To view, visit https://review.coreboot.org/21419
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I99cbcba7a086ef950f248888a83cf24a4db4aee9
Gerrit-Change-Number: 21419
Gerrit-PatchSet: 1
Gerrit-Owner: Patrick Georgi <pgeorgi at google.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20170906/c3db5546/attachment.html>


More information about the coreboot-gerrit mailing list