From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bruce Richardson Subject: [PATCH v2] scripts: add additional git log capitalization checks Date: Tue, 5 Jul 2016 17:03:11 +0100 Message-ID: <1467734591-24973-1-git-send-email-bruce.richardson@intel.com> References: <1467723939-30635-1-git-send-email-bruce.richardson@intel.com> Cc: thomas.monjalon@6wind.com, Bruce Richardson To: dev@dpdk.org Return-path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id 885D7689B for ; Tue, 5 Jul 2016 18:03:29 +0200 (CEST) In-Reply-To: <1467723939-30635-1-git-send-email-bruce.richardson@intel.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Add API, NUMA, HW, SW, FW and VMDq to list of words to capitalize properly. Since VMDq is a bit unusual, add it as special case check so we can print an error message giving the correct way to write it. Signed-off-by: Bruce Richadson --- v2: add NUMA to list --- scripts/check-git-log.sh | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/scripts/check-git-log.sh b/scripts/check-git-log.sh index 7d2c7ee..0f2b4bc 100755 --- a/scripts/check-git-log.sh +++ b/scripts/check-git-log.sh @@ -89,11 +89,14 @@ bad=$(echo "$headlines" | grep --color=always \ bad=$(echo "$headlines" | grep -E --color=always \ -e '\<(rx|tx|RX|TX)\>' \ -e '\<[pv]f\>' \ + -e '\<[hsf]w\>' \ -e '\' \ + -e ':.*\' \ -e ':.*\' \ -e ':.*\' \ -e ':.*\' \ -e ':.*\' \ + -e ':.*\' \ -e ':.*\' \ -e ':.*\' \ -e ':.*\' \ @@ -106,6 +109,12 @@ bad=$(echo "$headlines" | grep -E --color=always \ | sed 's,^,\t,') [ -z "$bad" ] || printf "Wrong headline lowercase:\n$bad\n" +# special case check for VMDq to give good error message +bad=$(echo "$headlines" | grep -E --color=always \ + -e '\<(vmdq|VMDQ)\>' \ + | sed 's,^,\t,') +[ -z "$bad" ] || printf "Wrong headline capitalization, use 'VMDq':\n$bad\n" + # check headline length (60 max) bad=$(echo "$headlines" | awk 'length>60 {print}' | sed 's,^,\t,') [ -z "$bad" ] || printf "Headline too long:\n$bad\n" -- 2.5.5