From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: AS31976 209.132.180.0/23 X-Spam-Status: No, score=-3.6 required=3.0 tests=AWL,BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,RCVD_IN_DNSWL_HI,RP_MATCHES_RCVD shortcircuit=no autolearn=ham autolearn_force=no version=3.4.0 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by dcvr.yhbt.net (Postfix) with ESMTP id BE66820450 for ; Thu, 2 Nov 2017 07:22:43 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752549AbdKBHWl (ORCPT ); Thu, 2 Nov 2017 03:22:41 -0400 Received: from cloud.peff.net ([104.130.231.41]:43668 "HELO cloud.peff.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1750907AbdKBHWk (ORCPT ); Thu, 2 Nov 2017 03:22:40 -0400 Received: (qmail 16810 invoked by uid 109); 2 Nov 2017 07:22:39 -0000 Received: from Unknown (HELO peff.net) (10.0.1.2) by cloud.peff.net (qpsmtpd/0.94) with SMTP; Thu, 02 Nov 2017 07:22:39 +0000 Authentication-Results: cloud.peff.net; auth=none Received: (qmail 18832 invoked by uid 111); 2 Nov 2017 07:22:49 -0000 Received: from sigill.intra.peff.net (HELO sigill.intra.peff.net) (10.0.0.7) by peff.net (qpsmtpd/0.94) with SMTP; Thu, 02 Nov 2017 03:22:49 -0400 Authentication-Results: peff.net; auth=none Received: by sigill.intra.peff.net (sSMTP sendmail emulation); Thu, 02 Nov 2017 03:22:37 -0400 Date: Thu, 2 Nov 2017 03:22:37 -0400 From: Jeff King To: Stefan Beller Cc: Philip Oakley , Junio C Hamano , Christian Couder , git , Ben Peart , Jonathan Tan , Nguyen Thai Ngoc Duy , Mike Hommey , Lars Schneider , Eric Wong , Christian Couder Subject: Re: On the nature of cover letters [WAS Re: [PATCH 0/6] Create Git/Packet.pm] Message-ID: <20171102072237.wbmrzrgfyxdb3m3v@sigill.intra.peff.net> References: <20171101071422.c2k4plhntlgpdnbk@sigill.intra.peff.net> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org On Wed, Nov 01, 2017 at 03:31:42PM -0700, Stefan Beller wrote: > On Wed, Nov 1, 2017 at 9:42 AM, Stefan Beller wrote: > > >> So it may make more sense just to cross-reference those merges with the > >> topics that spawned them on the mailing list. I.e., instead of copying > >> the cover letter contents, just record the message-id (and update it > >> whenever a new iteration of a topic is picked up via "git am"). That > >> lets you get the cover letter information _and_ see any discussion > >> or review around the patch. > > > > That sounds good. > > Actually I just found out about `am.messageId`, which adds the individual > message id as a footer. Maybe that is good enough? (Though it would > clutter every commit, not just the merge commits) It also means digging around to find the apex of the thread (though generally that can be done automatically with sufficiently smart tooling; I think public-inbox can do it pretty easily). I also like the idea that I could read "log --first-parent" to get an overview of the topics (with links). But probably associating a message id with each patch smooths out a lot of corner cases (it sidesteps the "where do you store it until the commit is made" question, and it works when there's no cover letter). And it gives enough hint for other software to figure out everything else. If the clutter is too much, it could also go into a git-notes ref (that's not already implemented, but it seems like it would be pretty easy to teach "git am" to do that). For a while, Thomas Rast used a script to heuristically create that mapping via git-notes after the fact. But if "git am" just did it automatically on behalf of Junio, that would be more robust. I will admit that I found I didn't use the mapping generated by Thomas's script all that much. But I do keep a local mailing list index and often search for the commit's subject as a mail subject, which roughly accomplishes the same thing. -Peff