From mboxrd@z Thu Jan 1 00:00:00 1970 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) Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga102.fm.intel.com with ESMTP; 05 Jul 2016 07:41:08 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.26,580,1459839600"; d="scan'208";a="1016030366" Received: from irvmail001.ir.intel.com ([163.33.26.43]) by fmsmga002.fm.intel.com with ESMTP; 05 Jul 2016 07:41:08 -0700 Received: from sivswdev01.ir.intel.com (sivswdev01.ir.intel.com [10.237.217.45]) by irvmail001.ir.intel.com (8.14.3/8.13.6/MailSET/Hub) with ESMTP id u65Ef6pn022124; Tue, 5 Jul 2016 15:41:06 +0100 Received: from sivswdev01.ir.intel.com (localhost [127.0.0.1]) by sivswdev01.ir.intel.com with ESMTP id u65Ef6O1000968; Tue, 5 Jul 2016 15:41:06 +0100 Received: (from bricha3@localhost) by sivswdev01.ir.intel.com with id u65Ef6L3000964; Tue, 5 Jul 2016 15:41:06 +0100 From: Bruce Richardson To: dev@dpdk.org Cc: thomas.monjalon@6wind.com, Bruce Richardson Date: Tue, 5 Jul 2016 15:40:50 +0100 Message-Id: <1467729650-925-1-git-send-email-bruce.richardson@intel.com> X-Mailer: git-send-email 1.7.4.1 Subject: [dpdk-dev] [PATCH] scripts: add check for net driver names without net prefix X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 05 Jul 2016 14:41:09 -0000 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