[Buildroot] [RFC] Effective Patchwork Use and Patch Review

[Thomas Petazzoni]

Dear Ryan Barnett,

On Mon, 16 Sep 2013 09:36:44 -0500, Ryan Barnett wrote:

> > Agreed. When I wrote that patchwork improved the situation, I meant
> > that patchwork provided a good overview of outstanding patches, both
> > for reviewers as for maintainers.
> 
> OK that fine. I was just wondering if anyone had used patchworks
> to write some scripts to pull patches down and then mark-up and send 
> the review back out over email. My email client is a pain for this so
> I was just wondering if there was a way to not use it...

No, the only person using scripts for patchwork is really the
maintainer, so that when he applies patches, they are automatically
marked as "Applied" on the patchwork.

Also, since I (and a few other core developers) have some kind of admin
access to patchwork, we can mark patches as Superseded, Rejected, etc.
But we do this with the web interface.

> Under Section 10.2 of the manual, would it be useful to update this 
> say explicitly state that the review takes place over the mailing 
> list? Currently I think the section implies it that but doesn't
> say it explicitly. 

Sure, if anything is unclear in the manual, it's certainly worth making
it explicit.

Best regards,

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com