Re: Make Paul Thomas a reviewer?

[Steve Kargl <sgk at troutmask dot apl dot washington dot edu>]

On Sun, May 29, 2005 at 06:45:01PM +0200, Paul Thomas wrote:
> 
> Perhaps the best thing would be to that submitters ask for somebody in 
> particular to do the review?  Then that reviewer has the opportunity to say 
> "no", they are too busy or that it will have to wait until xxxx/xx/xx.  The 
> submitter than has the option to try somebody else - again, on the list so 
> that it is evident to all what is happening.  It seems to me that this is a 
> circumstance where there is no shame in saying "no".  We are all 
> volunteers, after all, and sometimes we have "to get a life".
> 

Paul, I think your last sentence is the key.  "We are all volunteers",
so it will be difficult to assign patches for review and expect timely
resolution.

If you go to http://gcc.gnu.org/fortran/ and scroll down a bit,
you find a list of people who can approve patches: Paul Brook,
Steven Bosscher, Toon Moene, Tobias Schlueter, Bud Davis, 
Janne Blomqvist, and Steve Kargl.

I think we (the gfortran community/developers) would simply like
to add you to the list of reviewers.  Others that should be added
include Thomas Koening, FX, and Jerry DeLisle.  The four of you
have demonstrated a working knowledge of parts of the gfortran
internals by the quality (and quantity) of patches submitted.

Tobi's motivation of nominating you is your demonstrated abilities
to deal with the trans-* files.  I find patches to these files to
be difficult to review due to my limitations in understand the
tree-ssa stuff.  AFAIK, only Tobi and Paul B review these patches,
and if Tobi submits a patch to the trans-* stuff that leaves only
(busy) person as a possible reviewer.

-- 
Steve