review tools (was Re: Code review required for commits - formal Team vote.)

[simo]

On Wed, 2012-10-17 at 15:14 +0200, David Disseldorp wrote:
> On Wed, 17 Oct 2012 08:53:19 -0400
> Ira Cooper <ira at samba.org> wrote:
> 
> > Note: I don't care if it is Gerrit, or just a set of python scripts on
> > sn that say "this e-mail topic is active... this one is being reviewed
> > by x, this one is closed."
> > 
> > Nobody says this has to be complex.  It as complex as we want to make
> > it, I explicitly stuck to giving requirements for a reason.  How to
> > implement those requirements in a non-odious way is another topic. :)
> 
> I've used patchworks[1] in the past and can recommend it. It is non-
> intrusive such that the ML remains authoritative, and people can ignore
> it if they so choose.
> 
> Git + a mailing list should remain the only mandatory review tools IMO.
> 
> Cheers, David
> 
> 1) http://jk.ozlabs.org/projects/patchwork/

Sound cool, can you briefly describe how it works ?

Simo.

-- 
Simo Sorce
Samba Team GPL Compliance Officer <simo at samba.org>
Principal Software Engineer at Red Hat, Inc. <simo at redhat.com>