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: AS54825 139.178.88.0/22 X-Spam-Status: No, score=-3.7 required=3.0 tests=AWL,BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,RCVD_IN_DNSWL_MED, SPF_HELO_NONE,SPF_PASS shortcircuit=no autolearn=ham autolearn_force=no version=3.4.6 Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org [139.178.88.99]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by dcvr.yhbt.net (Postfix) with ESMTPS id C15361F44D for ; Wed, 10 Apr 2024 00:27:36 +0000 (UTC) Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sv.mirrors.kernel.org (Postfix) with ESMTPS id 3A40A283208 for ; Wed, 10 Apr 2024 00:27:36 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 7A519A59; Wed, 10 Apr 2024 00:27:28 +0000 (UTC) Received: from mail-qk1-f181.google.com (mail-qk1-f181.google.com [209.85.222.181]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9A5A039B for ; Wed, 10 Apr 2024 00:27:25 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.222.181 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1712708847; cv=none; b=anz8wMscjpry7AM9Z6+s4M36DJ/8jNdfAFjlslhx2rTsWSy7BZUePA1EKCBCb2ICu+8EKqAaRTqai7Y3t6Nscm5I7qn/pd/g6KHJG8mZ8fr6WQoAL1XnllRCvF5YQACbuxIqUXC1ugkp6cDedcP2L63dUhjdjDSKlJTHudk0kaU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1712708847; c=relaxed/simple; bh=OSDknPZEbpDhi+Tpm24y8pkEPu8KRPc/dquomTlf6Js=; h=MIME-Version:References:In-Reply-To:From:Date:Message-ID:Subject: To:Cc:Content-Type; b=EtATFBmUUccAFZECjEDlL/Nisg97t0Vdz+/yCOsH3cyLbj8ErWZak4VJ1Ln9ayZtMKcOAYMuYM8A9aSGOeknMnP+rcc5C49h03Q6PWUKGzyHy3Ysrc7HtTmlbD4LaAis+ExhzrbPmRhwl29Ki2ejnMYzJ23pJfe56sWWsxLhpic= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=sunshineco.com; spf=pass smtp.mailfrom=gmail.com; arc=none smtp.client-ip=209.85.222.181 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=sunshineco.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Received: by mail-qk1-f181.google.com with SMTP id af79cd13be357-78d57bd577dso277527685a.2 for ; Tue, 09 Apr 2024 17:27:25 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1712708844; x=1713313644; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=PkKkgnbmYkqR8JFhIrWw+6QwKQnZOaZj1NFnzFaxiio=; b=IaDGrQ/TM25/YqFbR/FJhJBo+hvTfmqmgKVypkggr0E3wSZiLo2Q6wU2flBvhkZTDo PHf4k7DXEqtVOcddK3SVOyWTV3cGMGTGenj5FT5aJUZsXIn3ZXw8KN4yqoGSgWxkMBC7 9/DYGfFOD4WJFQWsMaJHXSlyEuH7PHfzFy2PV00b/vAZqW8lgmtIeac0fdJaGK5L6k/g RxeJIhvLJPwVsbfEjukDbH1ItMJS9Khjmmb3JHRcVmLM56T8QexpeN1uZgASBL/OU7YK Rs6IfhT/4vMscZLdCqspawBCYVwlbhzPetLXMVShvIkuRJzuHb+K6/QLAjNHiJBw+Iqc M9RQ== X-Gm-Message-State: AOJu0YzU0sVhYkNNhkiPvvvZhmuKWx+o55aFOh/CXwO4MIrJ2TwCdI5Q LypfoclJ18gbMturtXZMKMSetb/bak8ry1EZsSYGL1JAJzbxRuOc/QtndqDmuYn9g+cc/Ov6yio K4Dg9/FN36ELGufw5HtDxtuQcEfE= X-Google-Smtp-Source: AGHT+IFkRBGaZB7PdefQr2uas6qoOLXRo799UTSPyaJKpxOw0mT0Eq8akBs+lkE/t7d7DyIiqbw/j1cKrBbWFVLfAqY= X-Received: by 2002:a05:6214:5090:b0:69b:1d31:f670 with SMTP id kk16-20020a056214509000b0069b1d31f670mr1367923qvb.13.1712708844492; Tue, 09 Apr 2024 17:27:24 -0700 (PDT) Precedence: bulk X-Mailing-List: git@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 References: <6f71b1731f2aed9c2f4dc101bf4349344b575d73.1712699815.git.gitgitgadget@gmail.com> In-Reply-To: <6f71b1731f2aed9c2f4dc101bf4349344b575d73.1712699815.git.gitgitgadget@gmail.com> From: Eric Sunshine Date: Tue, 9 Apr 2024 20:27:13 -0400 Message-ID: Subject: Re: [PATCH v3 5/8] SubmittingPatches: discuss reviewers first To: Linus Arver via GitGitGadget Cc: git@vger.kernel.org, Junio C Hamano , Johannes Schindelin , Jonathan Tan , Emily Shaffer , Patrick Steinhardt , Matthieu Moy , Linus Arver Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Tue, Apr 9, 2024 at 5:57=E2=80=AFPM Linus Arver via GitGitGadget wrote: > No matter how well someone configures their email tooling, understanding > who to send the patches to is something that must always be considered. > So discuss it first instead of at the end. > > In the following commit we will clean up the (now redundant) discussion > about sending security patches to the Git Security mailing list. > > Signed-off-by: Linus Arver > --- > diff --git a/Documentation/SubmittingPatches b/Documentation/SubmittingPa= tches > @@ -397,6 +397,36 @@ letter. > +After the list reached a consensus that it is a good idea to apply the > +patch, re-send it with "To:" set to the maintainer{current-maintainer} > +and "cc:" the list{git-ml} for inclusion. This is especially relevant > +when the maintainer did not heavily participate in the discussion and > +instead left the review to trusted others. This isn't a new problem since you're merely relocating this text (thus, very likely may be outside the scope of this series), but is this recommendation still accurate? Although I'm unable to locate it in the mailing list, I have some vague recollection of Junio relatively recently wondering why a patch series had been resent with no changes and why he had been made the direct To: recipient. It turned out that the author was following the above instructions. Generally speaking, Junio is quite good at picking up a patch series without the author having to follow these instructions to resend a patch series with no changes other than the To: header, so such instructions place unnecessary burden upon both submitters as well as reviewers (who have to spend extra cycles wondering why a series was rerolled and whether any changes were made). It would probably be more helpful (and less wasteful of reviewer time) to instruct the patch submitter to monitor "What's Cooking" and Junio's "seen" branch, and to ping the list (after a week or two) if the patch series hasn't been picked up or seen any response. > +Do not forget to add trailers such as `Acked-by:`, `Reviewed-by:` and > +`Tested-by:` lines as necessary to credit people who helped your > +patch, and "cc:" them when sending such a final version for inclusion. Again, not a new problem introduced by this patch, but it seems like all of these are actively wrong. In every case, these trailers are _given_ by reviewers _after_ a series has been submitted (thus, too late for the author to add them), and Junio typically is the one who latches the Reviewed-by:, Acked-by:, etc. by adding the trailer to the patches already in his tree. Instead of the above, much more useful trailers that a patch author can add are Helped-by: and Reported-by:.