Code review required for commits - Discuss.

[Volker Lendecke]

On Fri, Oct 12, 2012 at 11:31:38AM +0200, Kai Blin wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> On 2012-10-12 10:20, Volker Lendecke wrote:
> 
> > Just be a PITA for the reviewers. Send mails, call them, start to
> > use other means like hop on a train to Goettingen with a LART tool
> > :-)
> 
> That's a reasonable effort for people who work on Samba on their
> dayjob. I've got to take at least one day off at work to get to
> Göttingen. On the worst case scenario, that limits me to 30 patchsets
> a year, assuming I spend all my vacation getting patches in. ;)
> 
> I'm still worried that this system is going to leak patches,
> especially if we don't allow for pushes after patches were ignored for
> a while.
> 
> I think if we go towards a required review system, we really, really
> need to make sure we don't drop patches.

This comes down to people interested in it. To me this is a
social process that will have leaks, but all human activity
has the potential for flaws. I think the mechanism to fix
things after mistakes have happened don't need to be
formalized. If your patches for the DNS server are ignored
forever, the ultimate fix is to create a modular interface
so that you can do your own thing independently. This is
really the ultima ratio with many, many steps before that.

Volker

-- 
SerNet GmbH, Bahnhofsallee 1b, 37081 Göttingen
phone: +49-551-370000-0, fax: +49-551-370000-9
AG Göttingen, HRB 2816, GF: Dr. Johannes Loxen
http://www.sernet.de, mailto:kontakt at sernet.de