[Spice-devel] [PATCH spice-protocol] build-sys: simplify autogen

Alon Levy alon at pobox.com
Fri Dec 5 14:14:18 PST 2014


On 12/06/2014 12:00 AM, Jonathon Jongsma wrote:
> On Fri, 2014-12-05 at 23:41 +0200, Alon Levy wrote:
>> [snip]
>>>
>>> At the same time, I'm not sure mailing lists are the right tool for code
>>> review.  It's difficult to track which patches have been reviewed and
>>> which haven't.
>>
>> http://patchwork.freedesktop.org/project/Spice/list/ can help, linked
>> from the wiki btw (http://www.spice-space.org/page/Main_Page)
>>
>> (not getting into this otherwise :)
>>
> 
> Thanks, I had forgotten about that. I notice now that all patches listed
> on that site are in state "New". Is there a magical incantation we can
> insert into our review that will change the patch state to "Approved" or
> "Needs Work" or something? If so, can we start using this incantation
> rather than our traditional "ACK", so that patchwork will be able to
> track the true state of these patches?

I don't know that magic, but it does sound handy.

> 
> Jonathon
> 
> 



More information about the Spice-devel mailing list