Re: [PATCH] Enforce GMP/MPFR version requires and fix fortran/28276,27021

[Mike Stump <mrs at apple dot com>]

On Sep 25, 2006, at 2:12 PM, Ian Lance Taylor wrote:
> I believe that it would help to create a new class of maintainers:
> people who are permitted to approve patches in certain areas, but are
> not permitted to commit patches in those areas without approval from
> some other maintainer.

Internally we do exactly this at Apple.  The general rule is, everyone  
needs patches to be reviewed, and everyone can so review patches.   
People review what they can, as they can.

I'd rather have this sort of class to help ensure adequate reviewer  
bandwidth than to simply go without if the SC is reluctant to appoint  
any additional reviewers.

My feeling is that the project would be benefitted by additional  
reviewer bandwidth.  More bandwidth means the work load is lessoned  
for existing reviewers and leaves each reviewer more time to spend on  
reviews (if they so choose) and/or allows them the latitude to focus  
reviews into smaller areas of code.  We can use the blanket people  
more and more as architects and master planners to help the day-to-day  
reviewers with direction and guidance.

Also, I we can squeeze more work out of people by making the process  
more efficient.  :-)  More reviewers I think would help make the  
process more efficient.