From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bruce Richardson Subject: [PATCH] scripts: add check for net driver names without net prefix Date: Tue, 5 Jul 2016 15:40:50 +0100 Message-ID: <1467729650-925-1-git-send-email-bruce.richardson@intel.com> Cc: thomas.monjalon@6wind.com, Bruce Richardson To: dev@dpdk.org Return-path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by dpdk.org (Postfix) with ESMTP id 359D85689 for ; Tue, 5 Jul 2016 16:41:09 +0200 (CEST) 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" Since the current standard for commit prefixes is to put the device type e.g. "net", in front of the driver name, add a check for commit titles which have a net driver name without any prefix in front of it. Signed-off-by: Bruce Richardson --- scripts/check-git-log.sh | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/scripts/check-git-log.sh b/scripts/check-git-log.sh index 7d2c7ee..7dc1fd8 100755 --- a/scripts/check-git-log.sh +++ b/scripts/check-git-log.sh @@ -78,6 +78,12 @@ bad=$(echo "$headlines" | grep --color=always \ | sed 's,^,\t,') [ -z "$bad" ] || printf "Wrong headline label:\n$bad\n" +# check headline label for net/ prefix if needed +bad=$(echo "$headlines" | grep -E --color=always \ + -e "^($(ls drivers/net | grep -v Makefile | tr '\n' '|')):" \ + | sed 's,^,\t,') +[ -z "$bad" ] || printf "Headline missing 'net/' prefix:\n$bad\n" + # check headline lowercase for first words bad=$(echo "$headlines" | grep --color=always \ -e '^.*[A-Z].*:' \ -- 2.5.5