From: Arif Khokar <arif_khokar@hotmail.com>
To: Eric Wong <e@80x24.org>,
Johannes Schindelin <Johannes.Schindelin@gmx.de>,
Arif Khokar <arif_khokar@hotmail.com>
Cc: "Jakub Narębski" <jnareb@gmail.com>, "Jeff King" <peff@peff.net>,
"Stefan Beller" <sbeller@google.com>,
"git@vger.kernel.org" <git@vger.kernel.org>,
"meta@public-inbox.org" <meta@public-inbox.org>
Subject: Re: Working with public-inbox.org [Was: [PATCH] rev-parse: respect core.hooksPath in --git-path]
Date: Thu, 25 Aug 2016 03:40:08 +0000 [thread overview]
Message-ID: <DM5PR17MB135301ADAAB495F143EE62CFD3ED0@DM5PR17MB1353.namprd17.prod.outlook.com> (raw)
In-Reply-To: <20160824184938.GB8578@whir>
On 08/24/2016 02:49 PM, Eric Wong wrote:
>> On Tue, 23 Aug 2016, Arif Khokar wrote:
>>> Given that public-inbox provides an NNTP interface, couldn't the ARTICLE
>>> <message-id> NNTP command be used to easily retrieve the messages in a
>>> given patch series (at least compared to POP or IMAP). Perhaps
>>> git-send-email could be modified to include the message-id value of each
>>> patch in the series that it sends to the mailing list and include it in
>>> the cover letter.
>
> I think that makes sense; perhaps an X-Git-Followups: header
> from send-email which lists the child Message-IDs the same way
> References: does for ancestors.
That sounds like a better idea compared to what I came up with
originally and it would be much easier to parse out of the downloaded
cover letter message headers as opposed to looking for
delimiters/keywords in the message body.
> (perhaps there's already a standardized header for listing children)
I don't recall ever seeing anything like that in any RFC or message
header I've read through. It's an interesting idea though.
> I thought about allowing a giant MIME message with all the
> patches attached, too but that won't work for a large patch
> series due to size limits along various SMTP hops.
I think the vast majority of SMTP servers allow several megabyte
messages though their configured policy. I can't speak for those who
use their own SMTP servers though. NNTP servers may limit an individual
message to a megabyte or less though.
> Compression might make spam filters unhappy, too.
Perhaps, but there should be more reliance on IP blacklists and DMARC as
a first line of defense against SPAM.
>>> Then a script could be written (i.e., git-download-patch) which could
>>> parse the cover letter message (specified using its message-id), and
>>> download all the patches in series, which can then be applied using
>>> git-am. This would in fact take the email client out of the equation in
>>> terms of saving patches.
>
> w3m -dump -dump_source nntp://<NNTP-server>/<Message-ID>
>
> ought to already work for news.gmane.org and news.public-inbox.org
That's interesting. I didn't know w3m was capable of that. But, given
the way you specified it, it doesn't show the article headers.
Replacing the -dump and -dump_source options with -dump_both would
display the headers as well.
> The Net::NNTP Perl module is a standard part of the Perl distro
> for many years, now (along with Net::SMTP), so that would not
> be a roadblock for implementing a custom downloader distributed
> with git.
I wrote a prototype one and included it in my response to Johannes in
<DM5PR17MB1353B99EBD5F4FD23360DD41D3ED0@DM5PR17MB1353.namprd17.prod.outlook.com>.
It could be used as a starting point in terms of making it easier to
download patches to apply with git-am (without having to rely on one's
MUA or the tooling around it).
next prev parent reply other threads:[~2016-08-25 3:40 UTC|newest]
Thread overview: 64+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-08-16 16:55 Working with public-inbox.org [Was: [PATCH] rev-parse: respect core.hooksPath in --git-path] Stefan Beller
2016-08-16 17:10 ` Junio C Hamano
2016-08-16 17:20 ` Jeff King
2016-08-16 17:54 ` Junio C Hamano
2016-08-16 17:22 ` Stefan Beller
2016-08-16 17:47 ` Junio C Hamano
2016-08-16 20:44 ` Eric Wong
2016-08-16 20:56 ` Eric Wong
2016-08-18 12:42 ` Johannes Schindelin
2016-08-18 20:49 ` Eric Wong
2016-08-18 21:41 ` Junio C Hamano
2016-08-19 15:18 ` Johannes Schindelin
2016-08-19 15:30 ` Johannes Schindelin
2016-08-19 16:55 ` Stefan Beller
2016-08-19 22:35 ` Eric Wong
2016-08-22 13:38 ` Johannes Schindelin
2016-08-22 19:21 ` Jeff King
2016-08-19 22:35 ` Eric Wong
2016-08-22 13:18 ` Johannes Schindelin
2016-08-22 18:05 ` Jakub Narębski
2016-08-25 13:21 ` Johannes Schindelin
2016-08-28 18:23 ` Jakub Narębski
2016-08-29 10:46 ` Johannes Schindelin
2016-08-22 22:55 ` Eric Wong
2016-08-25 12:58 ` Johannes Schindelin
2016-08-27 22:38 ` Jakub Narębski
2016-08-28 8:36 ` Working with public-inbox.org Johannes Schindelin
2016-08-28 11:41 ` Jakub Narębski
2016-08-29 5:35 ` Johannes Schindelin
2016-08-19 15:03 ` Working with public-inbox.org [Was: [PATCH] rev-parse: respect core.hooksPath in --git-path] Jeff King
2016-08-20 19:57 ` Jakub Narębski
2016-08-23 4:47 ` Arif Khokar
2016-08-24 15:34 ` Johannes Schindelin
2016-08-24 18:49 ` Eric Wong
2016-08-24 19:12 ` Jeff King
2016-08-24 19:27 ` Eric Wong
2016-08-25 3:40 ` Arif Khokar [this message]
2016-08-25 2:41 ` Arif Khokar
2017-02-10 16:10 ` Johannes Schindelin
2017-02-13 5:52 ` Arif Khokar
2017-02-13 14:37 ` Johannes Schindelin
2017-02-14 3:56 ` Arif Khokar
2017-02-14 3:59 ` Arif Khokar
2017-02-14 7:13 ` Eric Wong
2017-02-13 19:21 ` Junio C Hamano
2017-02-14 3:55 ` Arif Khokar
2017-02-14 4:41 ` Junio C Hamano
2017-02-14 5:09 ` Arif Khokar
2017-02-14 5:14 ` Jeff King
2016-08-22 13:06 ` Johannes Schindelin
2016-08-22 13:15 ` Duy Nguyen
2016-08-22 20:38 ` Philip Oakley
2016-08-24 13:04 ` Johannes Schindelin
2016-08-24 19:16 ` Eric Wong
2016-08-25 13:08 ` Johannes Schindelin
2016-08-25 3:57 ` Arif Khokar
2016-08-25 13:01 ` Johannes Schindelin
2016-08-25 23:14 ` Arif Khokar
2016-08-26 8:08 ` Johannes Schindelin
2016-08-27 22:26 ` Jakub Narębski
2016-08-28 8:38 ` Johannes Schindelin
2016-08-28 13:45 ` Announcing Git User's Survey 2016 [was: Working with public-inbox.org] Jakub Narębski
2016-09-09 13:06 ` Johannes Schindelin
2016-09-09 18:51 ` Jakub Narębski
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: http://public-inbox.org/README
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=DM5PR17MB135301ADAAB495F143EE62CFD3ED0@DM5PR17MB1353.namprd17.prod.outlook.com \
--to=arif_khokar@hotmail.com \
--cc=Johannes.Schindelin@gmx.de \
--cc=e@80x24.org \
--cc=git@vger.kernel.org \
--cc=jnareb@gmail.com \
--cc=meta@public-inbox.org \
--cc=peff@peff.net \
--cc=sbeller@google.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://80x24.org/public-inbox.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).