git@vger.kernel.org list mirror (unofficial, one of many)
 help / color / mirror / code / Atom feed
* Re: Patchwork now does the right thing with GGG submissions
       [not found] <20201012180034.tu4rmeoadhy5j4w7@chatter.i7.local>
@ 2020-10-12 19:38 ` Jonathan Nieder
  2020-10-12 20:36   ` Junio C Hamano
  2020-10-12 21:31   ` Konstantin Ryabitsev
  0 siblings, 2 replies; 4+ messages in thread
From: Jonathan Nieder @ 2020-10-12 19:38 UTC (permalink / raw)
  To: Konstantin Ryabitsev; +Cc: gitster, git

Konstantin Ryabitsev wrote:

> Just FYI, I made the necessary tweaks to patchwork to make it properly
> assign "From" for gitgitgadget submissions:
>
> https://patchwork.kernel.org/patch/11832689/

\o/ Thanks much for this.

For the curious, any links to the change behind the scenes that made
that happens?  Does it use Sender[1] or a different field?  Does the
change live in [2] or [3]?

Sincerely,
Jonathan

[1] https://lore.kernel.org/git/xmqqiml4bsra.fsf@gitster-ct.c.googlers.com/
[2] https://git.kernel.org/pub/scm/linux/kernel/git/mricon/korg-helpers.git/
[3] https://github.com/getpatchwork/patchwork

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Patchwork now does the right thing with GGG submissions
  2020-10-12 19:38 ` Patchwork now does the right thing with GGG submissions Jonathan Nieder
@ 2020-10-12 20:36   ` Junio C Hamano
  2020-10-12 21:27     ` Konstantin Ryabitsev
  2020-10-12 21:31   ` Konstantin Ryabitsev
  1 sibling, 1 reply; 4+ messages in thread
From: Junio C Hamano @ 2020-10-12 20:36 UTC (permalink / raw)
  To: Jonathan Nieder; +Cc: Konstantin Ryabitsev, git

Jonathan Nieder <jrnieder@gmail.com> writes:

> Konstantin Ryabitsev wrote:
>
>> Just FYI, I made the necessary tweaks to patchwork to make it properly
>> assign "From" for gitgitgadget submissions:
>>
>> https://patchwork.kernel.org/patch/11832689/
>
> \o/ Thanks much for this.
>
> For the curious, any links to the change behind the scenes that made
> that happens?  Does it use Sender[1] or a different field?  Does the
> change live in [2] or [3]?
>
> Sincerely,
> Jonathan

It is curious that I sometimes see you respond to Konstantin on this
list, but the message you are responding to is not found.  What is
going on?

A question to Konstantin is what is so special about GGG?  Did you
make patchwork to pay attention to the in-body "From:" and other
in-body header lines (in which case it benefits not just GGG but
everybody).  If you added a support to ignore everything before the
scissors that would deserve an extra gold star ;-)

For example, how does

https://lore.kernel.org/git/xmqqy2kwiimi.fsf@gitster.c.googlers.com/

appear in patchwork?

> [1] https://lore.kernel.org/git/xmqqiml4bsra.fsf@gitster-ct.c.googlers.com/
> [2] https://git.kernel.org/pub/scm/linux/kernel/git/mricon/korg-helpers.git/
> [3] https://github.com/getpatchwork/patchwork

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Patchwork now does the right thing with GGG submissions
  2020-10-12 20:36   ` Junio C Hamano
@ 2020-10-12 21:27     ` Konstantin Ryabitsev
  0 siblings, 0 replies; 4+ messages in thread
From: Konstantin Ryabitsev @ 2020-10-12 21:27 UTC (permalink / raw)
  To: Junio C Hamano; +Cc: Jonathan Nieder, git

On Mon, Oct 12, 2020 at 01:36:39PM -0700, Junio C Hamano wrote:
> >> Just FYI, I made the necessary tweaks to patchwork to make it 
> >> properly
> >> assign "From" for gitgitgadget submissions:
> >>
> >> https://patchwork.kernel.org/patch/11832689/
> >
> > \o/ Thanks much for this.
> >
> > For the curious, any links to the change behind the scenes that made
> > that happens?  Does it use Sender[1] or a different field?  Does the
> > change live in [2] or [3]?
> >
> > Sincerely,
> > Jonathan
> 
> It is curious that I sometimes see you respond to Konstantin on this
> list, but the message you are responding to is not found.  What is
> going on?

I initially didn't send it to git@vger, only to Jonathan. :) I don't 
mind, though -- I just wasn't sure if this was something others on the 
git list would care about.

> A question to Konstantin is what is so special about GGG?  Did you
> make patchwork to pay attention to the in-body "From:" and other
> in-body header lines (in which case it benefits not just GGG but
> everybody).  If you added a support to ignore everything before the
> scissors that would deserve an extra gold star ;-)

No, this is specifically for GGG mail, as it's a very specific 
well-understood case. The solution I have in place is to just pay 
attention to the in-body Form, but once we upgrade to patchwork 2.2.2 
tomorrow, we will also start paying attention to the X-Original-From 
header.

I'm not 100% sure if it's always correct to pay attention to the in-body 
From -- at least for the purposes of patchwork. Patchwork pays attention 
to "who submitted the series," not "who authored the commit," so making 
in-body From always be the "From" in patchwork would be wrong.

For the cases of GGG, we can ignore this difference, as we definitely 
don't want to pay attention to "who sent the patch" (it's always 
gitgitgadget@gmail.com). However, GGG can make the situation more 
correct if it starts adding an X-Original-From that would be the Github 
name and email of the person submitting the pull request. E.g.:

From: D. Eveloper via GitGitGadget <ggg@gmail.com>   <- DMARC From
X-Original-From: D. Eveloper <github@email.here.com> <- Patchwork From
Subject: [PATCH 1/10] ...

From: A. Nother-Person <foo@example.com>             <- Git From

> For example, how does
> 
> https://lore.kernel.org/git/xmqqy2kwiimi.fsf@gitster.c.googlers.com/
> 
> appear in patchwork?

It's a reply to another patch, so patchwork only processes it as a 
comment. Generally, patchwork doesn't see worrying about this thing as 
its job -- it just wants to let maintainers track incoming patches and 
series and download them in a format that is easy to apply using "git 
am". For that matter, that's how b4 approaches this as well.

-K

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Patchwork now does the right thing with GGG submissions
  2020-10-12 19:38 ` Patchwork now does the right thing with GGG submissions Jonathan Nieder
  2020-10-12 20:36   ` Junio C Hamano
@ 2020-10-12 21:31   ` Konstantin Ryabitsev
  1 sibling, 0 replies; 4+ messages in thread
From: Konstantin Ryabitsev @ 2020-10-12 21:31 UTC (permalink / raw)
  To: Jonathan Nieder; +Cc: gitster, git

On Mon, Oct 12, 2020 at 12:38:50PM -0700, Jonathan Nieder wrote:
> > Just FYI, I made the necessary tweaks to patchwork to make it 
> > properly
> > assign "From" for gitgitgadget submissions:
> >
> > https://patchwork.kernel.org/patch/11832689/
> 
> \o/ Thanks much for this.
> 
> For the curious, any links to the change behind the scenes that made
> that happens?  Does it use Sender[1] or a different field?  Does the
> change live in [2] or [3]?

See my other email explaining how it's implemented. As to where it lives 
-- it's implemented as a procmail rule:

:0
* ^From: .*<gitgitgadget@gmail\.com>
{
    :0 bw
    INBODYFROM=| formail -xfrom:

    :0 hfw
    * INBODYFROM ?? .
    | formail -I "From:$INBODYFROM"
}

Currently, we fully blow away the From:, but after the 2.2.2 upgrade 
we'll be setting it as X-Original-From if we don't find one already 
present.

It's my goal to start publishing these procmail rules as part of a git 
repo in the future, so you'll be able to see all rules that are applied 
to incoming mail before we pass it along for patchwork processing.

Best regards,
-K

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2020-10-12 21:31 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <20201012180034.tu4rmeoadhy5j4w7@chatter.i7.local>
2020-10-12 19:38 ` Patchwork now does the right thing with GGG submissions Jonathan Nieder
2020-10-12 20:36   ` Junio C Hamano
2020-10-12 21:27     ` Konstantin Ryabitsev
2020-10-12 21:31   ` Konstantin Ryabitsev

git@vger.kernel.org list mirror (unofficial, one of many)

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://public-inbox.org/git
	git clone --mirror http://ou63pmih66umazou.onion/git
	git clone --mirror http://czquwvybam4bgbro.onion/git
	git clone --mirror http://hjrcffqmbrq6wope.onion/git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V1 git git/ https://public-inbox.org/git \
		git@vger.kernel.org
	public-inbox-index git

Example config snippet for mirrors.
Newsgroups are available over NNTP:
	nntp://news.public-inbox.org/inbox.comp.version-control.git
	nntp://7fh6tueqddpjyxjmgtdiueylzoqt6pt7hec3pukyptlmohoowvhde4yd.onion/inbox.comp.version-control.git
	nntp://ie5yzdi7fg72h7s4sdcztq5evakq23rdt33mfyfcddc5u3ndnw24ogqd.onion/inbox.comp.version-control.git
	nntp://4uok3hntl7oi7b4uf4rtfwefqeexfzil2w6kgk2jn5z2f764irre7byd.onion/inbox.comp.version-control.git
	nntp://news.gmane.io/gmane.comp.version-control.git
 note: .onion URLs require Tor: https://www.torproject.org/

code repositories for project(s) associated with this inbox:

	https://80x24.org/mirrors/git.git

AGPL code for this site: git clone https://public-inbox.org/public-inbox.git