From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 08017C4167D for ; Thu, 2 Nov 2023 12:27:35 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229806AbjKBM1f (ORCPT ); Thu, 2 Nov 2023 08:27:35 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60280 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229645AbjKBM1f (ORCPT ); Thu, 2 Nov 2023 08:27:35 -0400 Received: from orbyte.nwl.cc (orbyte.nwl.cc [IPv6:2001:41d0:e:133a::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 0DDC4112 for ; Thu, 2 Nov 2023 05:27:31 -0700 (PDT) Received: from n0-1 by orbyte.nwl.cc with local (Exim 4.94.2) (envelope-from ) id 1qyWnG-0001j3-N2; Thu, 02 Nov 2023 13:27:26 +0100 Date: Thu, 2 Nov 2023 13:27:26 +0100 From: Phil Sutter To: Pablo Neira Ayuso Cc: Arturo Borrero Gonzalez , Jeremy Sowden , netfilter-devel@vger.kernel.org, fw@strlen.de Subject: Re: [RFC] nftables 1.0.6 -stable backports Message-ID: Mail-Followup-To: Phil Sutter , Pablo Neira Ayuso , Arturo Borrero Gonzalez , Jeremy Sowden , netfilter-devel@vger.kernel.org, fw@strlen.de References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: netfilter-devel@vger.kernel.org On Thu, Nov 02, 2023 at 12:34:14PM +0100, Pablo Neira Ayuso wrote: > On Thu, Oct 19, 2023 at 05:23:15PM +0200, Phil Sutter wrote: > > Kindly find attached my collect_backports.sh. I keep it in an unused > > sub-directory (~/git/nftables/stable_tooling), but it's not necessary. > > It creates $(dirname $0)/backports directory containing a list of > > potential backports for each tag in the range defined by the variables > > in the script's header. > > I have integrated the pending fixes identified by your script. I have > refreshed 1.0.6.y branch and pushed it out. Thanks for doing this! > If anyone find more candidates to be merged into 1.0.6, Let me know. My script found two missing ones. Not sure if they came in late or if you explicitly omitted them: - 8519ab031d8022999603a69ee9f18e8cfb06645d - f65b2d12236174d477c55e96c4027cd51185ba5e As you know, my script relies upon Fixes: tags. I use git-notes to help it here and there. This way I added extra Fixes: tags with correct hashes for: - 818cc223b052b9a3b0bc3fc28a4b7036b5898408 - 8a9f48515fb8f9aed0af04e05f4528aa0e32116f - b4c9900c895fd55788912d62063cf107a27b68e0 - b593378b9b2470213af1892053af519801053a7e (The list may very well be incomplete.) Another case for git-notes is missing Fixes: tags in fixing commits. They are important in two ways: - Without a Fixes: tag, the script won't find the backport candidate. - If backported manually, the script won't find potential follow-ups (fixes may have fixes, too). In 1.0.6.y, I see 33 commits having no Fixes: tag. Did you solve this locally with git-notes? Assuming history rewriting is allowed in the stable branch, one could update the backports with Fixes: tags as needed and write a custom collect_backports.sh which operates on the stable branch directly. An alternative might be to publish notes somehow. According to the man page, it seems possible to keep them in a branch but I have no idea how this works. Cheers, Phil