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

[Khem Raj]

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.

> 
> Regards,
> 
> -- 
> Martin 'JaMa' Jansa     jabber: Martin.Jansa at gmail.com



> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel at lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel


-- 
-Khem