From giacomo@apache.org Sat Nov 4 07:56:33 2000 Return-Path: Mailing-List: contact cocoon-dev-help@xml.apache.org; run by ezmlm Delivered-To: mailing list cocoon-dev@xml.apache.org Received: (qmail 48151 invoked from network); 4 Nov 2000 07:56:33 -0000 Received: from stargazer2.dataway.ch (HELO dataway.ch) (195.216.80.34) by locus.apache.org with SMTP; 4 Nov 2000 07:56:33 -0000 Received: (qmail 25932 invoked from network); 4 Nov 2000 07:56:32 -0000 X-dataway-Spamcheck: 195.216.80.151: RBL=- DUL=- RSS=- ORBS=- Received: from unknown (HELO pwr.ch) (195.216.80.151) by stargazer.dataway.ch with SMTP; 4 Nov 2000 07:56:32 -0000 Received: (qmail 30459 invoked from network); 4 Nov 2000 07:13:07 -0000 Received: from donald.pwr.ch (HELO apache.org) (10.20.30.103) by simba.pwr.ch with SMTP; 4 Nov 2000 07:13:07 -0000 Sender: giacomo Message-ID: <3A03B702.5B19B0B1@apache.org> Date: Sat, 04 Nov 2000 08:13:06 +0100 From: Giacomo Pati Organization: Apache Software Foundation X-Mailer: Mozilla 4.74 [en] (X11; U; Linux 2.2.16 i686) X-Accept-Language: de-DE, de-CH, en MIME-Version: 1.0 To: cocoon-dev@xml.apache.org Subject: Re: [ADMIN] process for applying patches References: Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit X-Spam-Rating: locus.apache.org 1.6.2 0/1000/N Jeff Turner wrote: > > I think this is a general problem with popular open source projects. Each > project has to get the committer:contributor ratio right. Assuming Cocoon > has the right ratio, but still experiences problems... > > A possible solution: ordinary cocoon-devvers "register" themselves as > interested in certain parts of Cocoon. If you a) understand the general > architecture, and b) understand what file X.java does, register your > interest. When someone proposes a patch to X.java, those registered can > review the patch. If the patch looks good, reviewers +1 it on this list. > When a real committer has a chance to review the patch, they can have much > more confidence in the quality of the patch. Committers can concentrate on > whether a patch violates design issues, rather than simpler implementation > issues. My personal opinion about this is that I feel it is too formal. I think commiters and developers should automatically feel responsable for parts of the system they are interested on. And if a patch gets forgotten I hope those interested poeple stand up and say so on the list (and not only the original contributor). Nobody is perfect but if we help each other we can make it more perfect as a community not as an individual. > To implement this system, we could set up a web site where people register > themselves as reviewers, and indicate which files they are interested in. > Then if I modify X.java, I can look up who's registered as reviewers and > send the patch to them (in addition to cocoon-dev). There can be an online > voting system, where reviewers who do their job well get recognised > (possibly leading to commit access), and those who don't get automatically > removed after a few -1's. The whole system can work without any > intervention from the cocoon maintainers. What do others think about such a formalism? Giacomo