[Incubator] GRASS (and) PSC

Daniel Brookshier dbrookshier at collab.net
Fri May 5 10:26:54 EDT 2006

Sorry for the delay. I had a bunch of email get past me when I tried  
to compartmentalize it. I love to hate email :o)

jxta.org uses this process and it has done reasonably well. I have  
seen dozens of patches that would have messed things up, but were  
caught by review.

It is indeed a bit onerous to some, but it has its benefits.

As to "small improvements" those can be covered under a single  
artifact and review. Sometimes too, there is no need for a review,  
like adding docs or some types of refactoring.

The management of patches is a bit different with svn. It is real  
easy to create a patch branch that can be reviewed and tested. Lot  
easier looking at a diff between branches in a tool than a diff patch  

Daniel Brookshier | Community Manager | CollabNet, Inc.
8000 Marina Blvd. Suite 600 | Brisbane, CA 94005 | USA
O 972.422.5261 | C 214.207.6614 | dbrookshier at collab.net

On Apr 26, 2006, at 11:29 AM, Frank Warmerdam wrote:

> Daniel Brookshier wrote:
>> Here is a recommended way of running the process that is patterned  
>> after several successful projects: http://wiki.osgeo.org/index.php/ 
>> Managing_Changes_to_Code
> Daniel,
> I am curious if any of the existing OSGeo projects follow a process  
> like the
> one above.  With mapserver there has been some question whether we  
> should be
> creating bugzilla entries for every code change but resisters like  
> me have
> kept this from being mandated.  I *feel* like it would make it hard  
> to make
> small improvements.
> Also, do folks really browse through patches to see if they seem  
> sensible?
> I find patches very hard going as a rule for manual review.
> Also, generally for mapserver we only do RFC's and PSC votes for  
> "significant
> changes" where significant is loosely defined in RFC-1.
> Best regards,
> -- 
> --------------------------------------- 
> +--------------------------------------
> I set the clouds in motion - turn up   | Frank Warmerdam,  
> warmerdam at pobox.com
> light and sound - activate the windows | http://pobox.com/~warmerdam
> and watch the world go round - Rush    | President OSGF, http:// 
> osgeo.org
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: incubator-unsubscribe at incubator.osgeo.org
> For additional commands, e-mail: incubator-help at incubator.osgeo.org

More information about the Incubator mailing list