From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <dev-bounces@dpdk.org>
Received: from dpdk.org (dpdk.org [92.243.14.124])
	by inbox.dpdk.org (Postfix) with ESMTP id 8FB4BA052E;
	Mon,  3 Feb 2020 17:00:18 +0100 (CET)
Received: from [92.243.14.124] (localhost [127.0.0.1])
	by dpdk.org (Postfix) with ESMTP id 66BF81BFB4;
	Mon,  3 Feb 2020 17:00:18 +0100 (CET)
Received: from us-smtp-1.mimecast.com (us-smtp-delivery-1.mimecast.com
 [205.139.110.120]) by dpdk.org (Postfix) with ESMTP id C06B81BFAE
 for <dev@dpdk.org>; Mon,  3 Feb 2020 17:00:16 +0100 (CET)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com;
 s=mimecast20190719; t=1580745616;
 h=from:from:reply-to:subject:subject:date:date:message-id:message-id:
 to:to:cc:cc:mime-version:mime-version:content-type:content-type:
 content-transfer-encoding:content-transfer-encoding:
 in-reply-to:in-reply-to:references:references;
 bh=tsWRBgZLNfcAuuHguoGsbWN1bHfbmPsK4fECEcFuCuA=;
 b=LuG+yM3xRukoiuZ4fxYbffrqalQgY8wS3XjJG1bGu48AFkNYJIHzSnV8K0magL8mHU54Jy
 hHsGVpHIebNtaBuJyiLBAJgW/UhCf6F/QZsOf8fE7chk2BkzdSVU0WVE0vCeyxSJX9y9ce
 fD2dnX2FD0A6nhiFjau7jve9+No/oMw=
Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com
 [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id
 us-mta-426-IK3yKQ8GMa65CdUP1wL1kg-1; Mon, 03 Feb 2020 11:00:10 -0500
Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com
 [10.5.11.23])
 (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits))
 (No client certificate requested)
 by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 787EF1882CD2;
 Mon,  3 Feb 2020 16:00:09 +0000 (UTC)
Received: from rh.redhat.com (unknown [10.33.36.79])
 by smtp.corp.redhat.com (Postfix) with ESMTP id C9DCC19486;
 Mon,  3 Feb 2020 16:00:07 +0000 (UTC)
From: Kevin Traynor <ktraynor@redhat.com>
To: dev@dpdk.org,
	thomas@monjalon.net
Cc: stable@dpdk.org,
	bluca@debian.org,
	Kevin Traynor <ktraynor@redhat.com>
Date: Mon,  3 Feb 2020 15:59:55 +0000
Message-Id: <20200203155955.3949-1-ktraynor@redhat.com>
In-Reply-To: <20200203154426.3488-1-ktraynor@redhat.com>
References: <20200203154426.3488-1-ktraynor@redhat.com>
MIME-Version: 1.0
X-Scanned-By: MIMEDefang 2.84 on 10.5.11.23
X-MC-Unique: IK3yKQ8GMa65CdUP1wL1kg-1
X-Mimecast-Spam-Score: 0
X-Mimecast-Originator: redhat.com
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: quoted-printable
Subject: [dpdk-dev] [PATCH v2] devtools: add fixes column to git-log-fixes
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <https://mails.dpdk.org/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://mails.dpdk.org/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <https://mails.dpdk.org/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>

During backporting, if the fixes or stable tag are missing,
it usually requires some investigation by stable maintainer
as to why.

The presence of a fixes tag may be known from whether the
originating release of the issue is printed at the end of the
line but with variable line lengths and nested partial fixes
it doesn't catch the eye.

When there are a large amount of commits, adding an aligned column
indicating the presence of a fixes tag beside the stable one makes
it easier to quickly see the patches requiring further investigation.

e.g.
20.02 8f33cbcfa S F net/i40e/base: fix buffer address (16.04)
20.02 4b3da9415 S F net/i40e/base: fix error message (1.7.0)
20.02 1da546c39 - F net/i40e/base: fix missing link modes (17.08)
20.02 79bfe7808 S F net/i40e/base: fix Tx descriptors number (1.7.0)
20.02 50126939c - F net/i40e/base: fix retrying logic (18.02)
20.02 dcd05da0a S F app/testpmd: fix GENEVE flow item (18.02)
20.02 b0b9fdad2 S - net/bnx2x: support secondary process (N/A)
20.02 f8279f47d S F net/netvsc: fix crash in secondary process (18.08)

Signed-off-by: Kevin Traynor <ktraynor@redhat.com>

---
v2: reword commit msg

Not sure if anyone else is relying on this script and changing the
output will break something for them? If it is not ok to make this
change now, I will add a note and re-submit for 20.05.
---
 devtools/git-log-fixes.sh | 16 ++++++++++++++--
 1 file changed, 14 insertions(+), 2 deletions(-)

diff --git a/devtools/git-log-fixes.sh b/devtools/git-log-fixes.sh
index e37ee2260..6d468d673 100755
--- a/devtools/git-log-fixes.sh
+++ b/devtools/git-log-fixes.sh
@@ -95,9 +95,21 @@ stable_tag () # <hash>
 }
=20
+# print a marker for fixes tag presence
+fixes_tag () # <hash>
+{
+        if git log --format=3D'%b' -1 $1 | grep -qi '^Fixes: *' ; then
+                echo 'F'
+        else
+                echo '-'
+        fi
+}
+
 git log --oneline --reverse $range |
 while read id headline ; do
 =09origins=3D$(origin_filter $id)
 =09stable=3D$(stable_tag $id)
-=09[ "$stable" =3D "S" ] || [ -n "$origins" ] || echo "$headline" | grep -=
q fix || continue
+=09fixes=3D$(fixes_tag $id)
+=09[ "$stable" =3D "S" ] || [ "$fixes" =3D "F" ] || [ -n "$origins" ] || \
+=09=09echo "$headline" | grep -q fix || continue
 =09version=3D$(commit_version $id)
 =09if [ -n "$origins" ] ; then
@@ -109,4 +121,4 @@ while read id headline ; do
 =09=09origver=3D'N/A'
 =09fi
-=09printf '%s %7s %s %s (%s)\n' $version $id $stable "$headline" "$origver=
"
+=09printf '%s %7s %s %s %s (%s)\n' $version $id $stable $fixes "$headline"=
 "$origver"
 done
--=20
2.21.1