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