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

[Khem Raj]

On (18/10/11 23:42), Martin Jansa wrote:
> 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
> 

sorry about that glitch should be ok now.

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

Once registered it does not let the user change it.