httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jim Jagielski <...@jaguNET.com>
Subject Re: svn commit: r567091 - in /httpd/httpd/trunk: CHANGES modules/ldap/util_ldap.c
Date Fri, 17 Aug 2007 18:01:59 GMT

On Aug 17, 2007, at 1:46 PM, Eric Covener wrote:

> On 8/17/07, Jim Jagielski <jim@jagunet.com> wrote:
>> Does this change really require a CHANGES entry??
>>
>> covener@apache.org wrote:
>>>
>>> Author: covener
>>> Date: Fri Aug 17 10:33:11 2007
>>> New Revision: 567091
>>>
>>> URL: http://svn.apache.org/viewvc?view=rev&rev=567091
>>> Log:
>>> AFAICT, LDAP_CACHE_LOCK was a no-op when virtualhosts were used
>
> I can expand or remove it; there's crash/hang potential (observed) as
> the cache code goes into shared memory.
>

How about creating a PR about it and then closing it out.
That way it's a documented bug that is closed; if it
affects people, the PR will provide insight into what
was wrong and what was fixed, and the CHANGES entry
can be adjusted to reflect the PR number.


Mime
View raw message