Re: Patchwork now does the right thing with GGG submissions

From: Konstantin Ryabitsev <konstantin@linuxfoundation.org>
Date: Oct 12, 2020, 21:31 UTC
Message-ID: <20201012213107.ad3j2wetrfs62pvp@chatter.i7.local>
In-Reply-To: <20201012193850.GE3740546@google.com>

On Mon, Oct 12, 2020 at 12:38:50PM -0700, Jonathan Nieder wrote:

Show 11 quoted lines

> > 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