Re: Working with public-inbox.org [Was: [PATCH] rev-parse: respect core.hooksPath in --git-path]

From: Junio C Hamano <gitster@pobox.com>
Date: Feb 13, 2017, 19:21 UTC
Message-ID: <xmqqefz1ew1h.fsf@gitster.mtv.corp.google.com>
In-Reply-To: <d16546a4-25be-7b85-3191-e9393fda1164@hotmail.com>

Arif Khokar <arif.i.khokar@gmail.com> writes:

Show 5 quoted lines

> ... I
> still think it would be better to be able to list the message-id
> values in the header or body of the cover letter message of a patch
> series (preferably the former) in order to facilitate downloading the
> patches via NNTP from gmane or public-inbox.org.

You are looking at builtin/log.c::make_cover_letter()? Patches welcome, but I think you'd need a bit of preparatory refactoring to allow gen_message_id() be called for all messages _before_ this function is called, as currently we generate them as we emit each patch.

> Alternatively, or perhaps in addition to the list of message-ids, a
> list of URLs to public-inbox.org or gmane messages could also be
> provided for those who prefer to download patches via HTTP.

Many people around here do not want to repeat the mistake of relying too much on one provider. Listing Message-IDs may be a good idea, listing URLs that are tied to one provider is less so.