From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id AED3EA0548 for ; Fri, 3 Dec 2021 08:53:52 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 80D5840041; Fri, 3 Dec 2021 08:53:52 +0100 (CET) Received: from smtp-relay-internal-1.canonical.com (smtp-relay-internal-1.canonical.com [185.125.188.123]) by mails.dpdk.org (Postfix) with ESMTP id EDFC440041 for ; Fri, 3 Dec 2021 08:53:50 +0100 (CET) Received: from mail-qv1-f72.google.com (mail-qv1-f72.google.com [209.85.219.72]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by smtp-relay-internal-1.canonical.com (Postfix) with ESMTPS id AA3033F1B6 for ; Fri, 3 Dec 2021 07:53:50 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=canonical.com; s=20210705; t=1638518030; bh=Yzb4nOGOGNGOFjPdNwobEQwMcX1bQGNYxUtCEcrAH/4=; h=MIME-Version:References:In-Reply-To:From:Date:Message-ID:Subject: To:Content-Type; b=eMSbpExCgkmBxH2fPkkpylIh+yvLnNGO1LkRmKxXc/ABh5KIr2gSnC901Q/4RzJZr TdOrwh01EINx0QDeoWY13n+u+CQ/xEmtAz6sGl6LDNXj/3XFgnAt4pciABSoG6XV8p 3OLsMaG9Px6+0yNyzf/C2FEkKvApuvNzM9MbYI6VOryv6aXZFvqukX0k19WP0Uk/E1 a442a7VSJeXJg1BsIXrWrJgHY3vjVvkU1jEh0Q4OF0QvRHK9ZMvTpv1j4odRVbINi6 p6ldbyGuJSucSLNTUXksQ5Sp8ttWZm41J539NbGcMGGL90nFUL8qq+3FXKhKjzyMv0 h6QIFBNI8aKHw== Received: by mail-qv1-f72.google.com with SMTP id q2-20020a05621419e200b003aeeeff5417so2587968qvc.9 for ; Thu, 02 Dec 2021 23:53:50 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to; bh=Yzb4nOGOGNGOFjPdNwobEQwMcX1bQGNYxUtCEcrAH/4=; b=ObV0VooJzrtXY7MV7Kt7slbf4tgx9Nuw9ZtY+ODDvprKdQPA5xsxrgAnC9dTkarcbx Ki3bUuHFJ1eguQp7UKv2U4+Rr1RSWrLdzUhklcaOXM5/HHdvILsxvWLyc8G0Gix2ncax UDZB+ftM/No4zmg1QChGj5oWcQsBMtqyaOzWtltW/WhilUp9oVr56IjS4M1W9szApVBf TC5pUCbOs8ZpmPl8p45UlTVsWNfXrWldXDFX9ney4FtSJfZPHLw4vjmAGS6brqcefanj nvKkH+aKLKlEpqIQteanegRuC7acIKnm1dxLmuio4KzO8F0JIDsGuFPRXQbEcjw6RAD3 fRDA== X-Gm-Message-State: AOAM531VpHEBmSqvrOUSQhOuLG3xtuE24KMtb5Nsihy/OsL0sc7TAfsj H4UAxhBukpmpXAJPYrgods0PBtx2yzhMNlpDoO+jbOsI1fIXOHRq7CmBufD8YPA32XWs702XDC0 Pf/45zQyP98R2YH0k0btHsgHR6I662goFVY2ngF6C X-Received: by 2002:ad4:5762:: with SMTP id r2mr17756071qvx.31.1638518029739; Thu, 02 Dec 2021 23:53:49 -0800 (PST) X-Google-Smtp-Source: ABdhPJyLUd17lF1jtfblkvErf1LD4BkwlNDmwXC3a5XG60Zr/ZgMB58IaIC+JSmTMk4YK0jQre+DdpSshnNVCiADwJw= X-Received: by 2002:ad4:5762:: with SMTP id r2mr17756057qvx.31.1638518029497; Thu, 02 Dec 2021 23:53:49 -0800 (PST) MIME-Version: 1.0 References: <20211203075123.4081128-1-christian.ehrhardt@canonical.com> <20211203075123.4081128-2-christian.ehrhardt@canonical.com> In-Reply-To: <20211203075123.4081128-2-christian.ehrhardt@canonical.com> From: Christian Ehrhardt Date: Fri, 3 Dec 2021 08:53:23 +0100 Message-ID: Subject: Re: [PATCH 2/2] devtools: report commit id on partial fixes To: stable@dpdk.org, Thomas Monjalon , Luca Boccassi , Xueming Li , David Marchand , Kevin Traynor , Maxime Coquelin Content-Type: text/plain; charset="UTF-8" X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org On Fri, Dec 3, 2021 at 8:51 AM wrote: > > From: Christian Ehrhardt Hmm, that probably should go to dev@ instead since that is where the script lives - resubmitting > The candidate list for backports lists partial fixes like: > "(21.02 (partially fixed in 21.08))" > That is useful to identify fixes menat for later releases, but > indirectly applying to older ones as well. > > While the devscript has no access to the stable tree to fully check > if the respective interim commit is present there, reporting the commit > id will still help to check it later - because only if the interim commit > id is in the stable tree, then also the new fix is a real candidate > for backporting. > > The above would become "(21.02 (partially fixed in c30751afc360 @ 21.08))" > > Signed-off-by: Christian Ehrhardt > --- > devtools/git-log-fixes.sh | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/devtools/git-log-fixes.sh b/devtools/git-log-fixes.sh > index 27ec9088d4..a1b548948a 100755 > --- a/devtools/git-log-fixes.sh > +++ b/devtools/git-log-fixes.sh > @@ -76,7 +76,7 @@ origin_version () # ... > # look chained fix of fix recursively > local rootver="$(origin_version $roothashes)" > [ -n "$rootver" ] || continue > - echo "$rootver (partially fixed in $origver)" > + echo "$rootver (partially fixed in $origin @ $origver)" > else > echo "$origver" > fi > -- > 2.34.0 > -- Christian Ehrhardt Staff Engineer, Ubuntu Server Canonical Ltd