From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-f47.google.com (mail-wm0-f47.google.com [74.125.82.47]) by dpdk.org (Postfix) with ESMTP id DDEFB6CB1 for ; Wed, 6 Jul 2016 12:39:03 +0200 (CEST) Received: by mail-wm0-f47.google.com with SMTP id 187so31674128wmz.1 for ; Wed, 06 Jul 2016 03:39:03 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind-com.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=zfae1Az6oBSvZO8IA8NgfdUxVu/aRQxGYMl/rmXgQF8=; b=OKPVGfITS+OPGWQjcrrSD4iYbf6qKhURIDSREXMd2o402dgDHq2sjxe0HsdI2KgjfA xTLGpaIJP1M85gYKUScDkafxCaq1D6Bpul6DdbX9Bw3iAUPLSgozYY+JhQh9g4r9OXE4 9jKr+RF0mqGH2tZwvERvrJ6yvnR7byjcCp/E42TatRXe/NUCTOyGMufVlGoj6IqW/0Da gzJ8V97axs+/cXkBWl8buuWAlYfg0lXF+EyMlke9m/UnNtncz7QNtsmfLU29sQf30jB5 Pp1A/1/lIKX5gOcufIC3RNElW2PQRoqJ2xDMdZgPcY04viztYb+vJIT+h3z/cu62Sw/K wWfg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=zfae1Az6oBSvZO8IA8NgfdUxVu/aRQxGYMl/rmXgQF8=; b=lzVMoHWqXZV/5DXHPq/94QFX6/TOS+phDDtF4a/YNlFlZNzujLbz3YkShiK2iR7LP+ m5ZYke9IVJW5R4D8Lv+duVndTsBiojS5FQ8Jq1flehr/YmUFnQTU99xYs1xgyvRJQzdm 75323V+q7bTiIq9XQ5u6SzEyvDCu0y6RH8PsBkiiaAeh4vGuEbzaN5iQpgN62DF2koxR /iyrXymmnnI1nmC9qZyoRiMhRgRtSJaT+wjo9q9cdX3/jl84HhdwTfFDnRrqgfFUKfdB BAkaqy0kKN06hAmQljJvpQ4KfMZa0KbETVVjUt64Yr8Jx1BNwuGE04dJdvngnr+fg/Xf gf5A== X-Gm-Message-State: ALyK8tJ+INIl75KWQbHdthZ1tXHIx8Rlw8OVfocp4e7ljdNRbYx+Gf1SOu0z+dEQTmPKLS4C X-Received: by 10.194.192.38 with SMTP id hd6mr17805878wjc.129.1467801543450; Wed, 06 Jul 2016 03:39:03 -0700 (PDT) Received: from XPS13.localdomain (184.203.134.77.rev.sfr.net. [77.134.203.184]) by smtp.gmail.com with ESMTPSA id x194sm3381094wmf.13.2016.07.06.03.39.02 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Wed, 06 Jul 2016 03:39:02 -0700 (PDT) From: Thomas Monjalon To: dev@dpdk.org Cc: Nelio Laranjeiro Date: Wed, 6 Jul 2016 12:38:50 +0200 Message-Id: <1467801530-20800-4-git-send-email-thomas.monjalon@6wind.com> X-Mailer: git-send-email 2.7.0 In-Reply-To: <1467801530-20800-1-git-send-email-thomas.monjalon@6wind.com> References: <1467801530-20800-1-git-send-email-thomas.monjalon@6wind.com> Subject: [dpdk-dev] [PATCH 3/3] scripts: check headline of drivers commits 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: Wed, 06 Jul 2016 10:39:04 -0000 From: Nelio Laranjeiro A driver patch under net should start with "net/" or if a patch touch multiple drivers, it should only start with "net:". The same apply for crypto. A patch touching all drivers (net + crypto) should start with "drivers:". Longer prefixes like "net/mlx:" (for mlx4/mlx5) or "net/e1000/base:" are handled by not checking the colon. Signed-off-by: Nelio Laranjeiro Signed-off-by: Thomas Monjalon --- scripts/check-git-log.sh | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) diff --git a/scripts/check-git-log.sh b/scripts/check-git-log.sh index 833aa39..3f3cf19 100755 --- a/scripts/check-git-log.sh +++ b/scripts/check-git-log.sh @@ -70,6 +70,23 @@ bad=$(echo "$headlines" | grep --color=always \ | sed 's,^,\t,') [ -z "$bad" ] || printf "Wrong headline format:\n$bad\n" +# check headline prefix when touching only drivers/, e.g. net/ +bad=$(for commit in $commits ; do + headline=$(git log --format='%s' -1 $commit) + files=$(git diff-tree --no-commit-id --name-only -r $commit) + [ -z "$(echo "$files" | grep -v '^drivers/')" ] || continue + driversgrp=$(echo "$files" | cut -d "/" -f 2 | sort -u) + drivers=$(echo "$files" | cut -d "/" -f 2,3 | sort -u) + if [ $(echo "$driversgrp" | wc -l) -gt 1 ] ; then + bad=$(echo "$headline" | grep -v '^drivers:') + elif [ $(echo "$drivers" | wc -l) -gt 1 ] ; then + bad=$(echo "$headline" | grep -v "^$driversgrp") + else + bad=$(echo "$headline" | grep -v "^$drivers") + fi +done | sed 's,^,\t,') +[ -z "$bad" ] || printf "Wrong headline prefix:\n$bad\n" + # check headline label for common typos bad=$(echo "$headlines" | grep --color=always \ -e '^example[:/]' \ -- 2.7.0