You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by rp...@apache.org on 2007/12/20 22:49:54 UTC

svn commit: r606036 - /httpd/httpd/branches/2.2.x/STATUS

Author: rpluem
Date: Thu Dec 20 13:49:53 2007
New Revision: 606036

URL: http://svn.apache.org/viewvc?rev=606036&view=rev
Log:
* Add a proposal

Modified:
    httpd/httpd/branches/2.2.x/STATUS

Modified: httpd/httpd/branches/2.2.x/STATUS
URL: http://svn.apache.org/viewvc/httpd/httpd/branches/2.2.x/STATUS?rev=606036&r1=606035&r2=606036&view=diff
==============================================================================
--- httpd/httpd/branches/2.2.x/STATUS (original)
+++ httpd/httpd/branches/2.2.x/STATUS Thu Dec 20 13:49:53 2007
@@ -99,6 +99,14 @@
                Better IMHO to separate releasing it from the many bugfixes
                in 2.2.7 mod_proxy.  Hence +1 for 2.2.8.
 
+   * mod_disk_cache: Delete temporary files if they cannot be renamed to their
+     final name.
+      Trunk version of patch:
+         http://svn.apache.org/viewcvs.cgi?rev=409942&view=rev
+      Backport version for 2.2.x of patch:
+         Trunk version of patch works
+      +1: rpluem,
+
 PATCHES/ISSUES THAT ARE STALLED
 
    * beos MPM: Create pmain pool and run modules' child_init hooks when



Re: svn commit: r606036 - /httpd/httpd/branches/2.2.x/STATUS

Posted by Ruediger Pluem <rp...@apache.org>.

On 12/20/2007 11:25 PM, Nick Kew wrote:
> On Thu, 20 Dec 2007 21:49:54 -0000
> rpluem@apache.org wrote:
> 
> 
>> +   * mod_disk_cache: Delete temporary files if they cannot be
>> renamed to their
>> +     final name.
>> +      Trunk version of patch:
>> +         http://svn.apache.org/viewcvs.cgi?rev=409942&view=rev
>> +      Backport version for 2.2.x of patch:
>> +         Trunk version of patch works
>> +      +1: rpluem,
> 
> That's three errors that are basically the same (one new in the patch).
> Two are at APLOG_DEBUG, one at APLOG_ERR.  Why the difference?

Ask Paul (r190043, r190530) :-).
No seriously, I do not know, but good catch. They should use the same level.
What is your preference? DEBUG or ERR?

Regards

RĂ¼diger



Re: svn commit: r606036 - /httpd/httpd/branches/2.2.x/STATUS

Posted by Nick Kew <ni...@webthing.com>.
On Thu, 20 Dec 2007 21:49:54 -0000
rpluem@apache.org wrote:


> +   * mod_disk_cache: Delete temporary files if they cannot be
> renamed to their
> +     final name.
> +      Trunk version of patch:
> +         http://svn.apache.org/viewcvs.cgi?rev=409942&view=rev
> +      Backport version for 2.2.x of patch:
> +         Trunk version of patch works
> +      +1: rpluem,

That's three errors that are basically the same (one new in the patch).
Two are at APLOG_DEBUG, one at APLOG_ERR.  Why the difference?

-- 
Nick Kew

Application Development with Apache - the Apache Modules Book
http://www.apachetutor.org/