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/10/15 21:38:39 UTC

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

Author: rpluem
Date: Mon Oct 15 12:38:38 2007
New Revision: 584867

URL: http://svn.apache.org/viewvc?rev=584867&view=rev
Log:
* Add some comments.

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=584867&r1=584866&r2=584867&view=diff
==============================================================================
--- httpd/httpd/branches/2.2.x/STATUS (original)
+++ httpd/httpd/branches/2.2.x/STATUS Mon Oct 15 12:38:38 2007
@@ -190,6 +190,7 @@
      rewritten.
      jim says: Is rpluem's concern addressed?
      niq says: Yes, when we add r584842 to the proposal.
+     rpluem says: Yes.
      
    * mpm_winnt: Fix null pointer dereference
      PR 42572
@@ -203,6 +204,8 @@
      -1: jim: As per Roy, (forward) proxies should not check
               the URI. See r583466 and r583803.
      niq: updated proposal to address Roy & Jim's points.
+     rpluem: The first hunk of your patch fails (the include) because it is
+     already there. Remove this and I am +1.
 
    * core: backport ap_send_interim_response API
      http://svn.apache.org/viewvc?view=rev&revision=582630
@@ -220,6 +223,8 @@
      Once fixed I am +1.
      niq says: Do you mean my addition to the documentation included
      in r582631, or is there something else that needs reconciling?
+     rpluem says: Yes, there is a conflict in backporting the mod_proxy_http.xml
+     changes of r582631.
 
 PATCHES/ISSUES THAT ARE STALLED