Make Francois-Xavier Coudert, Thomas Koenig and Paul Thomas Reviewers?

Paul Thomas paulthomas2@wanadoo.fr
Mon Aug 8 16:36:00 GMT 2005


Paul,

Sorry about the delay in replying - yesterday, we had a huge fire in the 
forest behind our house, which resulted in our being evacuated 
overnight.  There is still firefighting going on and I have just had to 
give the firemen access across our land to connect to our irrigation 
water supply.

I had an exchange with Steven about this.  He voiced the opinion that 
the number of reviewers would become excessive and that this would 
reduce the confidence that other gcc maintainers have in gfortran; or in 
the control of the reviewing process.  I can see what he means but 
cannot see another way around progressing the work on gfortran when its 
reviewers are so pressed for time.  We should strive, therefore, to make 
sure that the reviewing is completely open.  In the course of our 
discussion, Steven opened my eyes to the existence of the patch queue.  
I honestly had no idea of its existence.  We must also make sure that it 
is used by all the contributors and the reviewers.

Paul Brook wrote:

>>I propose that Francois-Xavier Coudert, Thomas Koenig (if he is willing
>>- I have not discussed it with him) and I be made reviewers.  We are all
>>sensible enough that we can defer on contributions that are outside our
>>experience or competence.  In addition, since the reviewing process is
>>entirely open, there is really little prospect of disaster.  
>>    
>>
>
>Ok. Please update the webpages/docs to reflect this.
>  
>
Francois-Xavier; the pompier-sapeurs are back again - could you take 
care of this, please?

>>Perhaps, it 
>>would be a good idea, whilst we still have provisional licences, that 24
>>hours elapse before our giving a green light on the list and the
>>contributor committing, so that those with more experience can keep an
>>eye on what we are doing?
>>    
>>
>
>I don't think there's any need to formalise this. If you think this is 
>appropriate, just say so in your review mail. As always if you're unsure 
>about something, say so and either don't approve the patch, or make approval 
>conditional on someone else agreeing that the change is OK.
>  
>
OK

>Patch approvals should be driven by what you feel confident reviewing, not how 
>important the bugfix/feature it.
>  
>
Agreed.

Paul T



More information about the Fortran mailing list