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 > > _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/spice-devel