Re: [Spice-devel] Getting patchwork to acknowledge acks?

2017-08-07 Thread Uri Lublin
On 08/04/2017 03:52 PM, Christophe de Dinechin wrote: On 4 Aug 2017, at 14:49, Victor Toso wrote: On Fri, Aug 04, 2017 at 02:34:44PM +0200, Christophe de Dinechin wrote: If I look at a patch I sent, then I have a button and also a “delegate to” pop up. One of the

Re: [Spice-devel] Getting patchwork to acknowledge acks?

2017-08-04 Thread Christophe de Dinechin
> On 4 Aug 2017, at 14:49, Victor Toso wrote: > > On Fri, Aug 04, 2017 at 02:34:44PM +0200, Christophe de Dinechin wrote: >> >>> On 4 Aug 2017, at 14:03, Frediano Ziglio wrote: >>> Hi, On Fri, Aug 04, 2017 at 01:01:05PM +0200,

Re: [Spice-devel] Getting patchwork to acknowledge acks?

2017-08-04 Thread Victor Toso
On Fri, Aug 04, 2017 at 02:34:44PM +0200, Christophe de Dinechin wrote: > > > On 4 Aug 2017, at 14:03, Frediano Ziglio wrote: > > > >> > >> Hi, > >> > >> On Fri, Aug 04, 2017 at 01:01:05PM +0200, Christophe de Dinechin wrote: > It does not check if given patch was

Re: [Spice-devel] Getting patchwork to acknowledge acks?

2017-08-04 Thread Christophe de Dinechin
> On 4 Aug 2017, at 14:03, Frediano Ziglio wrote: > >> >> Hi, >> >> On Fri, Aug 04, 2017 at 01:01:05PM +0200, Christophe de Dinechin wrote: It does not check if given patch was pushed, no. :( >>> >>> If so, that’s a lot less useful than PRs. >> >> All comes down to

Re: [Spice-devel] Getting patchwork to acknowledge acks?

2017-08-04 Thread Frediano Ziglio
> > Hi, > > On Fri, Aug 04, 2017 at 01:01:05PM +0200, Christophe de Dinechin wrote: > > > It does not check if given patch was pushed, no. :( > > > > If so, that’s a lot less useful than PRs. > > All comes down to workflow in the end... Some people love it :) > https://patchwork.linuxtv.org/ >

Re: [Spice-devel] Getting patchwork to acknowledge acks?

2017-08-04 Thread Victor Toso
Hi, On Fri, Aug 04, 2017 at 01:01:05PM +0200, Christophe de Dinechin wrote: > > It does not check if given patch was pushed, no. :( > > If so, that’s a lot less useful than PRs. All comes down to workflow in the end... Some people love it :) https://patchwork.linuxtv.org/ signature.asc

Re: [Spice-devel] Getting patchwork to acknowledge acks?

2017-08-04 Thread Christophe de Dinechin
> On 4 Aug 2017, at 12:48, Victor Toso wrote: > > Hi, > > On Fri, Aug 04, 2017 at 12:42:39PM +0200, Christophe de Dinechin wrote: >> During the discussion on PRs and stuff, several people pointed to >> patchwork. >> >> Does anyone know why this tool does not acknowledge

Re: [Spice-devel] Getting patchwork to acknowledge acks?

2017-08-04 Thread Victor Toso
Hi, On Fri, Aug 04, 2017 at 12:42:39PM +0200, Christophe de Dinechin wrote: > During the discussion on PRs and stuff, several people pointed to > patchwork. > > Does anyone know why this tool does not acknowledge acks? For example > https://patchwork.freedesktop.org/series/27298/ has a "Acked-by:

[Spice-devel] Getting patchwork to acknowledge acks?

2017-08-04 Thread Christophe de Dinechin
During the discussion on PRs and stuff, several people pointed to patchwork. Does anyone know why this tool does not acknowledge acks? For example https://patchwork.freedesktop.org/series/27298/ has a "Acked-by: Christophe de Dinechin ” in the fifth comment, and has been