cvs commit: ports/net/asterisk Makefile ports/net/asterisk/files patch-main-utils.c patch-main::utils.c

[Pav Lucistnik]

Maxim Sobolev wrote:
> Jeremy Chadwick wrote:
>> The user and I were discussing, privately, scheduler-related things, and
>> the PR was mentioned.  I told him that ports maintainers are allowed up
>> to 2 weeks to respond, after which other committers can take over if
>> need be.  After 2 weeks had passed, the user provided me the patch (the
>> original PR mail snipped his attachment), and I committed it.
> 
> I still don't see any place where it says that the assigned PR with no 
> activity for more than 2 weeks on it should be considered as an approval 
> request. Just opening PR is not enough IMHO, it's usually task of the 
> requester to contact maintainer and seek for explicit approval if he 
> wants faster turnaround.

Maxim, you are out of touch with the common practice here as of lately. 
Indeed, we have been applying two-week timeouts on PRs assigned to 
committers, for a long time now. Generally, it works well. Obviously 
there are exceptions...

Pav