Re: Patchwork breaking submitters name Was: Patchwork hooks for meta-oe active and working

[Martin Jansa <martin.jansa@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 2611 bytes --]

On Tue, Oct 18, 2011 at 02:36:35PM -0700, Khem Raj wrote:
> On (18/10/11 23:10), Martin Jansa wrote:
> > On Tue, Oct 18, 2011 at 01:49:54PM -0700, Khem Raj wrote:
> > > On (18/10/11 22:38), Martin Jansa wrote:
> > > > On Mon, Oct 17, 2011 at 09:32:13AM +0200, Koen Kooi wrote:
> > > > > -----BEGIN PGP SIGNED MESSAGE-----
> > > > > Hash: SHA1
> > > > > 
> > > > > Hi,
> > > > > 
> > > > > Thanks to Khem the patchwork hooks for meta-oe (and hence meta-efl,
> > > > > meta-gnome and meta-xfce) are active and working.
> > > > > Once the commits hit the OE server the state will get updated automatically,
> > > > > so the submitter will instantly know when his/her patch has hit the repo.
> > > > > 
> > > > > regards,
> > > > 
> > > > There is known and fixed issue with patchwork breaking author name
> > > > 
> > > > We have an example in patch
> > > > "[meta-oe] lightmediascanner: Add version 0.4.4 (initial recipe)"
> > > > Original e-mail says "Pau Espin Pedrol <pespin.shar at gmail.com>"
> > > > http://lists.linuxtogo.org/pipermail/openembedded-devel/2011-October/035644.html
> > > > 
> > > > older patchwork instance says only "pespin <pespin.shar at gmail.com>"
> > > > http://patches.openembedded.org/patch/13323/
> > > > newer patchwork instance says also "pespin <pespin.shar at gmail.com>"
> > > > http://patchwork.openembedded.org/patch/13323/
> > > > 
> > > > Same problem was reported also on LKML
> > > > https://lkml.org/lkml/2011/9/18/128
> > > > And later fixed by Jeremy Kerr
> > > > https://lkml.org/lkml/2011/9/18/149
> > > > 
> > > > Can we get this upstream fix to our patchwork instance too?
> > > 
> > > This patch is already applied to the new patchwork instance on
> > > patchwork.openembedded.org however it was only yesterday when it came
> > > into life.
> > 
> > But still it doesn't work today:
> > http://patchwork.openembedded.org/patch/13437/
> > 
> > and from Kerr's reply I'm not sure if someone has to fix something in
> > patchwork database or if registered users can fix it in their profile
> > page.. or maybe he fixed only Submitter fields for existing patches in
> > database..
> 
> yes the user profile for existing users have to be updated which I have
> done for Pau now. So it should be ok. Please re-check the patchwork.

Almost :), there is missing 'l'
From: Pau Espin Pedro
From: Pau Espin Pedrol

Is it possible for registred patchwork user to update it or do we need
to ask you when we found similar issue for someone else?

Thanks!

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 205 bytes --]