From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: X-Spam-Status: No, score=-3.8 required=3.0 tests=AWL,BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS shortcircuit=no autolearn=ham autolearn_force=no version=3.4.6 Received: from out1.vger.email (out1.vger.email [IPv6:2620:137:e000::1:20]) by dcvr.yhbt.net (Postfix) with ESMTP id B714A1F428 for ; Fri, 10 Mar 2023 09:05:52 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231468AbjCJJFb (ORCPT ); Fri, 10 Mar 2023 04:05:31 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:41886 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231539AbjCJJEz (ORCPT ); Fri, 10 Mar 2023 04:04:55 -0500 Received: from cloud.peff.net (cloud.peff.net [104.130.231.41]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 0879AF2484 for ; Fri, 10 Mar 2023 01:01:25 -0800 (PST) Received: (qmail 19159 invoked by uid 109); 10 Mar 2023 09:01:24 -0000 Received: from Unknown (HELO peff.net) (10.0.1.2) by cloud.peff.net (qpsmtpd/0.94) with ESMTP; Fri, 10 Mar 2023 09:01:24 +0000 Authentication-Results: cloud.peff.net; auth=none Received: (qmail 10368 invoked by uid 111); 10 Mar 2023 09:01:24 -0000 Received: from coredump.intra.peff.net (HELO sigill.intra.peff.net) (10.0.0.2) by peff.net (qpsmtpd/0.94) with (TLS_AES_256_GCM_SHA384 encrypted) ESMTPS; Fri, 10 Mar 2023 04:01:24 -0500 Authentication-Results: peff.net; auth=none Date: Fri, 10 Mar 2023 04:01:24 -0500 From: Jeff King To: Junio C Hamano Cc: =?utf-8?B?w4Z2YXIgQXJuZmrDtnLDsA==?= Bjarmason , git@vger.kernel.org Subject: Re: [PATCH v2] gpg-interface: lazily initialize and read the configuration Message-ID: References: <230308.86fsafzllu.gmgdl@evledraar.gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org On Thu, Mar 09, 2023 at 09:03:35AM -0800, Junio C Hamano wrote: > Jeff King writes: > > > On Wed, Mar 08, 2023 at 09:34:15AM +0100, Ævar Arnfjörð Bjarmason wrote: > > > >> As added review: This is the same patch diff as I sent on February 9th: > >> https://lore.kernel.org/git/patch-1.2-d93c160dcbc-20230209T142225Z-avarab@gmail.com/; > >> my local range-diff to my previously submitted topic & next being: > >> [...] > >> So this LGTM. > > > > Thanks, and sorry for stealing your patch. I forgot that yours existed > > in that thread (and obviously I'm happy if either is applied). > > I am not Ævar but the last time this happened what he said was that > he did so not because he wanted to complain that somebody else stole > his thunder but because he wanted to show his agreement to the patch > by pointing at an indenendent invention of the same thing. > > I personally do not appreciate that tactics, exactly because it can > easily be misinterpreted as a complaint. Saying "I read the patch > and I think it is exactly how I would solve the problem, too. > Looking good" would have been much safer in that regard and conveyed > the same agreement. I think you are being too hard on Ævar here. While I agree it can be interpreted as passive aggressive sniping, I'd be a little frustrated, too, if I had written a patch and then somebody submitted the exact same thing later. (Hence my response that it was "oops" and not an intentional slight). If two people are independently doing the same work, we're wasting effort, and it's worth thinking about how we can avoid that. In this particular case, my opinion is that Ævar's original patch was OK by itself, but it was coupled with an unwelcome change. Submitting the cleanup to the now-empty callbacks on their own would have had a greater chance of success. (As a maintainer, you can also split up patches after the fact, but there is cognitive load in doing so. Minimizing that load is something submitters can do to help the project scale). -Peff