[dpdk-dev] scripts: fix commit check for empty list

Message ID 1468000231-13468-1-git-send-email-thomas.monjalon@6wind.com (mailing list archive)
State Accepted, archived
Headers

Commit Message

Thomas Monjalon July 8, 2016, 5:50 p.m. UTC
  When running check-git-log.sh on a clean tree it was complaining
of a wrong empty headline because '^[^:]*$' was matching.
It is fixed by matching at least one character with +.

Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>
---
 scripts/check-git-log.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Thomas Monjalon July 10, 2016, 2:35 p.m. UTC | #1
2016-07-08 19:50, Thomas Monjalon:
> When running check-git-log.sh on a clean tree it was complaining
> of a wrong empty headline because '^[^:]*$' was matching.
> It is fixed by matching at least one character with +.
> 
> Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>

Applied
  

Patch

diff --git a/scripts/check-git-log.sh b/scripts/check-git-log.sh
index 7d2c7ee..c2a3ac0 100755
--- a/scripts/check-git-log.sh
+++ b/scripts/check-git-log.sh
@@ -62,7 +62,7 @@  bad=$(echo "$headlines" | grep --color=always \
 	-e '\.$' \
 	-e '[,;!?&|]' \
 	-e ':.*_' \
-	-e '^[^:]*$' \
+	-e '^[^:]\+$' \
 	-e ':[^ ]' \
 	-e ' :' \
 	| sed 's,^,\t,')