You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by kk...@apache.org on 2010/10/13 18:18:49 UTC

svn commit: r1022171 - /tomcat/tc6.0.x/trunk/STATUS.txt

Author: kkolinko
Date: Wed Oct 13 16:18:48 2010
New Revision: 1022171

URL: http://svn.apache.org/viewvc?rev=1022171&view=rev
Log:
vote and proposal

Modified:
    tomcat/tc6.0.x/trunk/STATUS.txt

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=1022171&r1=1022170&r2=1022171&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Wed Oct 13 16:18:48 2010
@@ -214,7 +214,7 @@ PATCHES PROPOSED TO BACKPORT:
   http://localhost:8080/examples/jsp/snp;x=y/snoop.jsp
   http://people.apache.org/~markt/patches/2010-10-06-path-param-tc6.patch
   +1: markt
-  -1:
+  -0: kkolinko: see Re:r1005192 thread on dev@
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=49965
   Use correct i18n resources for StringManager in JAASRealm
@@ -222,6 +222,15 @@ PATCHES PROPOSED TO BACKPORT:
   (Port of http://svn.apache.org/viewvc?view=revision&revision=920820)
   +1: markt
   -1:
+  -0: kkolinko: It is amazing that the patch just removes N lines and fixes
+    the issue, but I think that removing static protected fields is not
+    good for binary compatibility between releases. Alternative patch
+    proposed below.
+
+  Alternative patch:
+  http://people.apache.org/~kkolinko/patches/2010-10-13_tc6_bug49965.patch
+  +1: kkolinko
+  -1:
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=49811
   Add context option to disable URL re-writing and session parsing from URLs



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org