You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by Marion & Christophe JAILLET <ch...@wanadoo.fr> on 2017/02/17 18:49:08 UTC

Re: svn commit: r1783413 - in /httpd/httpd/branches/2.4.x: ./ STATUS server/mpm/event/event.c

No CHANGES entry?

CJ


Le 17/02/2017 � 16:36, jim@apache.org a �crit :
> Author: jim
> Date: Fri Feb 17 15:36:02 2017
> New Revision: 1783413
>
> URL: http://svn.apache.org/viewvc?rev=1783413&view=rev
> Log:
> Merge r1774541 from trunk:
>
> event: close a race condition where we might re-enable listeners while they
> are already or about to be closed.
>
>
> Submitted by: ylavic
> Reviewed by: ylavic, jim, wrowe
>
> Modified:
>      httpd/httpd/branches/2.4.x/   (props changed)
>      httpd/httpd/branches/2.4.x/STATUS
>      httpd/httpd/branches/2.4.x/server/mpm/event/event.c

Re: svn commit: r1783413 - in /httpd/httpd/branches/2.4.x: ./ STATUS server/mpm/event/event.c

Posted by Luca Toscano <to...@gmail.com>.
My 2c! It would be great to see a little CHANGES entry, it might be useful
for people checking new mpm-features of the next httpd release (reviewing
the commit logs might be a bit overkill for some users).

Luca

2017-02-18 14:47 GMT+01:00 Jim Jagielski <ji...@jagunet.com>:

> CHANGES is usually for PR listing and significant user-land
> changes and not necessarily and full listing of all fixes...
> That's what the commit logs are for :)
>
> > On Feb 17, 2017, at 1:49 PM, Marion & Christophe JAILLET <
> christophe.jaillet@wanadoo.fr> wrote:
> >
> > No CHANGES entry?
> >
> > CJ
> >
> >
> > Le 17/02/2017 à 16:36, jim@apache.org a écrit :
> >> Author: jim
> >> Date: Fri Feb 17 15:36:02 2017
> >> New Revision: 1783413
> >>
> >> URL: http://svn.apache.org/viewvc?rev=1783413&view=rev
> >> Log:
> >> Merge r1774541 from trunk:
> >>
> >> event: close a race condition where we might re-enable listeners while
> they
> >> are already or about to be closed.
> >>
> >>
> >> Submitted by: ylavic
> >> Reviewed by: ylavic, jim, wrowe
> >>
> >> Modified:
> >>     httpd/httpd/branches/2.4.x/   (props changed)
> >>     httpd/httpd/branches/2.4.x/STATUS
> >>     httpd/httpd/branches/2.4.x/server/mpm/event/event.c
>
>

Re: svn commit: r1783413 - in /httpd/httpd/branches/2.4.x: ./ STATUS server/mpm/event/event.c

Posted by Jim Jagielski <ji...@jaguNET.com>.
CHANGES is usually for PR listing and significant user-land
changes and not necessarily and full listing of all fixes...
That's what the commit logs are for :)

> On Feb 17, 2017, at 1:49 PM, Marion & Christophe JAILLET <ch...@wanadoo.fr> wrote:
> 
> No CHANGES entry?
> 
> CJ
> 
> 
> Le 17/02/2017 à 16:36, jim@apache.org a écrit :
>> Author: jim
>> Date: Fri Feb 17 15:36:02 2017
>> New Revision: 1783413
>> 
>> URL: http://svn.apache.org/viewvc?rev=1783413&view=rev
>> Log:
>> Merge r1774541 from trunk:
>> 
>> event: close a race condition where we might re-enable listeners while they
>> are already or about to be closed.
>> 
>> 
>> Submitted by: ylavic
>> Reviewed by: ylavic, jim, wrowe
>> 
>> Modified:
>>     httpd/httpd/branches/2.4.x/   (props changed)
>>     httpd/httpd/branches/2.4.x/STATUS
>>     httpd/httpd/branches/2.4.x/server/mpm/event/event.c