From 4ff23a22462fa2637be45a771fd4a22fa7e3d4ad Mon Sep 17 00:00:00 2001 From: Nicholas Chin Date: Thu, 21 Mar 2024 11:27:50 -0600 Subject: util/lint: Fix license header regex A trailing "|" at the end of the regex added a zero length alternative match, causing all files to match and be filtered out. This was causing `make lint-stable` to ignore all missing license headers, preventing the pre-commit git hook and Jenkins from detecting these. Also, a missing "|" separator between cmos.default and .apcb would cause those files to be unintentionally scanned. Change-Id: I70cc3a5adf7edee059883cd3cbe02029776b02ef Signed-off-by: Nicholas Chin Reviewed-on: https://review.coreboot.org/c/coreboot/+/81422 Tested-by: build bot (Jenkins) Reviewed-by: Felix Singer --- util/lint/lint-000-license-headers | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'util') diff --git a/util/lint/lint-000-license-headers b/util/lint/lint-000-license-headers index 68bdae7bad..56addac710 100755 --- a/util/lint/lint-000-license-headers +++ b/util/lint/lint-000-license-headers @@ -67,8 +67,8 @@ _shipped$|\ \.spd|\ config|\ cmos\.layout|\ -cmos\.default\ -\.apcb$|\ +cmos\.default|\ +\.apcb$\ " #space separated list of directories to test -- cgit v1.2.3