Message ID | 1467801530-20800-4-git-send-email-thomas.monjalon@6wind.com (mailing list archive) |
---|---|
State | Superseded, archived |
Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [IPv6:::1]) by dpdk.org (Postfix) with ESMTP id 662636CB5; Wed, 6 Jul 2016 12:39:05 +0200 (CEST) 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 <dev@dpdk.org>; Wed, 6 Jul 2016 12:39:03 +0200 (CEST) Received: by mail-wm0-f47.google.com with SMTP id 187so31674128wmz.1 for <dev@dpdk.org>; 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 <thomas.monjalon@6wind.com> To: dev@dpdk.org Cc: Nelio Laranjeiro <nelio.laranjeiro@6wind.com> 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 <dev.dpdk.org> List-Unsubscribe: <http://dpdk.org/ml/options/dev>, <mailto:dev-request@dpdk.org?subject=unsubscribe> List-Archive: <http://dpdk.org/ml/archives/dev/> List-Post: <mailto:dev@dpdk.org> List-Help: <mailto:dev-request@dpdk.org?subject=help> List-Subscribe: <http://dpdk.org/ml/listinfo/dev>, <mailto:dev-request@dpdk.org?subject=subscribe> Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org> |
Commit Message
Thomas Monjalon
July 6, 2016, 10:38 a.m. UTC
From: Nelio Laranjeiro <nelio.laranjeiro@6wind.com> A driver patch under net should start with "net/<driver name>" 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 <nelio.laranjeiro@6wind.com> Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com> --- scripts/check-git-log.sh | 17 +++++++++++++++++ 1 file changed, 17 insertions(+)
Comments
On Wed, Jul 06, 2016 at 12:38:50PM +0200, Thomas Monjalon wrote: > From: Nelio Laranjeiro <nelio.laranjeiro@6wind.com> > > A driver patch under net should start with "net/<driver name>" 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 <nelio.laranjeiro@6wind.com> > Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com> > --- > 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/<driver name> > +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 Given the preference for including documentation in with the code changes, we should perhaps exclude any doc changes when making this check. /Bruce
Thomas, On Wed, Jul 06, 2016 at 12:38:50PM +0200, Thomas Monjalon wrote: > From: Nelio Laranjeiro <nelio.laranjeiro@6wind.com> > > A driver patch under net should start with "net/<driver name>" 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 <nelio.laranjeiro@6wind.com> > Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com> > --- > 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/<driver name> > +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:') I think you forgot the '$' for drivers in the line just above. > + 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 Otherwise it seems good. Regards,
2016-07-06 12:09, Bruce Richardson: > On Wed, Jul 06, 2016 at 12:38:50PM +0200, Thomas Monjalon wrote: > > From: Nelio Laranjeiro <nelio.laranjeiro@6wind.com> > > +# check headline prefix when touching only drivers/, e.g. net/<driver name> > > +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 > > Given the preference for including documentation in with the code changes, > we should perhaps exclude any doc changes when making this check. Yes we can exclude doc/ and config/. Thanks
2016-07-06 13:59, Nélio Laranjeiro: > > +# check headline prefix when touching only drivers/, e.g. net/<driver name> > > +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:') > > I think you forgot the '$' for drivers in the line just above. No it is a plain "drivers:" for cases where we have net and crypto. (it is an addition from your original idea) But I forgot to remove bad=$( which I've moved to the first line. > > + 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" > > Otherwise it seems good. V2 on going
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/<driver name> +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[:/]' \