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 E0509C61D92 for ; Tue, 21 Nov 2023 12:45:28 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232764AbjKUMpa (ORCPT ); Tue, 21 Nov 2023 07:45:30 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:37708 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232638AbjKUMp3 (ORCPT ); Tue, 21 Nov 2023 07:45:29 -0500 Received: from ganesha.gnumonks.org (ganesha.gnumonks.org [IPv6:2001:780:45:1d:225:90ff:fe52:c662]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id BCA03138 for ; Tue, 21 Nov 2023 04:45:25 -0800 (PST) Received: from [78.30.43.141] (port=53794 helo=gnumonks.org) by ganesha.gnumonks.org with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1r5Q81-007jeq-9j; Tue, 21 Nov 2023 13:45:23 +0100 Date: Tue, 21 Nov 2023 13:45:20 +0100 From: Pablo Neira Ayuso To: Thomas Haller Cc: Phil Sutter , NetFilter Subject: Re: [PATCH nft 1/1] tests/shell: sanitize "handle" in JSON output Message-ID: References: <20231117171948.897229-1-thaller@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: netfilter-devel@vger.kernel.org Hi Thomas, On Tue, Nov 21, 2023 at 01:10:11PM +0100, Thomas Haller wrote: > On Sat, 2023-11-18 at 03:36 +0100, Phil Sutter wrote: > > On Fri, Nov 17, 2023 at 06:18:45PM +0100, Thomas Haller wrote: [...] > > > Note that only a few .json-nft files are adjusted, because > > > otherwise the > > > patch is too large. Before applying, you need to adjust them all, > > > by > > > running `./tests/shell/run-tests.sh -g`. > > > > Just put the bulk change into a second patch? > > it would require 3 patches to stay below the limit. > > Also, it blows up the inbox by everybody on the list by 850K (57k > gzipped). The rest of the patch is generated. Just generate it. Excuse for jumping through this, just a submission style notice: Please, Cc maintainers with big patches that don't fit in into the mailing list in the future. > Alternatively, > > git fetch https://gitlab.freedesktop.org/thaller/nftables df984038a33c6da5b159e6f6458351c4fa673bf1 > git merge FETCH_HEAD I'd suggest you place this also in the cover letter, so at least there is a record on the mailing list archive that some patches could not get through but there was an alternative way to take them for others. As for maintainers, they can collect the large patch as attachment from the email. Thanks.