Helping in reviewing

Thiago Macieira thiago.macieira at intel.com
Tue Mar 26 14:06:43 PDT 2013


On terça-feira, 26 de março de 2013 19.32.54, Bernhard Friedreich wrote:
> What I've been wondering lately (and also in the light of recent events) is
> if there is a specific reason why there's no CI (Jenkins?) and/or
> reviewboard system (Gerrit?). Is it because reviewing patches on the
> maillinglist gathers more feedback (out of previous experiences) or just
> because it is more comfortable for you? Is it more comfortable? I guess
> some system like gerrit scales better.. Or is it because no one set up
> something yet?
> 
> Has there been thought about setting up a code quality system? For example
> "sonar" is very good at those things and is widely used - at least in the
> java world. Didn't try it with C/C++ yet..

This is really up to the people doing reviews. They have to choose a method 
that is most comfortable to them. In this aspect, "convenience for the casual 
drive-by-submitter" is completely trumped by "convenience for the reviewer".

I can't make that decision.

-- 
Thiago Macieira - thiago.macieira (AT) intel.com
  Software Architect - Intel Open Source Technology Center
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 190 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20130326/5f35b292/attachment.pgp>


More information about the wayland-devel mailing list