Patchwork hooks for meta-oe active and working

Patchwork hooks for meta-oe active and working

[Koen Kooi]

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

Koen
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (Darwin)
Comment: GPGTools - http://gpgtools.org

iD8DBQFOm9n9MkyGM64RGpERAoUuAJ90LE+rFXpLSeqiAe8F294u6E9mDQCeKGGq
iHdn71yB6/+siFiM2fjvERg=
=fqWk
-----END PGP SIGNATURE-----

Re: Patchwork hooks for meta-oe active and working

[Denys Dmytriyenko]

On Mon, Oct 17, 2011 at 09:32:13AM +0200, Koen Kooi wrote:
> 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.

Great! Thanks a lot for such a nice feature!

-- 
Denys

Re: Patchwork hooks for meta-oe active and working

[Anders Darander]

* Koen Kooi <koen@dominion.thruhere.net> [111017 09:33]:
> 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.

Thanks!
That sounds wonderfull!

Cheers,
Anders

-- 
Anders Darander
ChargeStorm AB / eStorm AB

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

[Martin Jansa]

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

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?

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

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

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

[Khem Raj]

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.

-Khem

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

[Martin Jansa]

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

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

Regards,

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

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

Re: 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@gmail.com



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


-- 
-Khem

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

[Martin Jansa]

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

Re: 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.