devtools: fix letter case check in commit title
authorThomas Monjalon <thomas@monjalon.net>
Mon, 18 Oct 2021 09:55:58 +0000 (11:55 +0200)
committerThomas Monjalon <thomas@monjalon.net>
Fri, 22 Oct 2021 20:40:59 +0000 (22:40 +0200)
The prefix (before the colon) of the title is lowercase.
The check of uppercase/lowercase in the commit title
was supposed to apply after the colon,
but some greps were not limited to the exact word.
So in the case of "test/dma: add basic dmadev instance tests",
the lowercase word "dmadev" was wrongly suggested to be uppercase.

The words of the dictionary must be filtered as whole word
with the grep option -w.

Fixes: d448efa259e9 ("devtools: export dictionary for commit title check")
Cc: stable@dpdk.org
Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
devtools/check-git-log.sh

index 9988bf8..885d444 100755 (executable)
@@ -111,12 +111,12 @@ IFS='
 '
 words="$selfdir/words-case.txt"
 for word in $(cat $words); do
-       bad=$(echo "$headlines" | grep -iw $word | grep -v $word)
+       bad=$(echo "$headlines" | grep -iw $word | grep -vw $word)
        if [ "$word" = "Tx" ]; then
                bad=$(echo $bad | grep -v 'OCTEON\ TX')
        fi
        for bad_line in $bad; do
-               bad_word=$(echo $bad_line | cut -d":" -f2 | grep -io $word)
+               bad_word=$(echo $bad_line | cut -d":" -f2 | grep -iwo $word)
                [ -z "$bad_word" ] || { printf "Wrong headline case:\n\
                        \"$bad_line\": $bad_word --> $word\n" && failure=true;}
        done