You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by re...@apache.org on 2008/05/17 00:59:03 UTC

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

Author: remm
Date: Fri May 16 15:59:02 2008
New Revision: 657244

URL: http://svn.apache.org/viewvc?rev=657244&view=rev
Log:
- Votes.

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=657244&r1=657243&r2=657244&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Fri May 16 15:59:02 2008
@@ -59,11 +59,12 @@
 
 * Fix NPE in patch for bug 44986
   http://svn.apache.org/viewvc?rev=657149&view=rev
-  +1: markt
+  +1: markt, remm
   -1: 
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=45015
   You can't use an unescaped quote if you quote the value with that character
   http://svn.apache.org/viewvc?rev=657231&view=rev
   +1: markt
-  -1: 
+  -1: remm (I think it is a good patch, but it throws an error for something which was working before,
+            so I would think keeping it for the next release would be better)



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


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

Posted by Mark Thomas <ma...@apache.org>.
remm@apache.org wrote:
> Author: remm
> Date: Fri May 16 15:59:02 2008
> New Revision: 657244
> 
> URL: http://svn.apache.org/viewvc?rev=657244&view=rev
> Log:
> - Votes.
> 
> 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=657244&r1=657243&r2=657244&view=diff
> ==============================================================================
> --- tomcat/tc6.0.x/trunk/STATUS.txt (original)
> +++ tomcat/tc6.0.x/trunk/STATUS.txt Fri May 16 15:59:02 2008
> @@ -59,11 +59,12 @@
>  
>  * Fix NPE in patch for bug 44986
>    http://svn.apache.org/viewvc?rev=657149&view=rev
> -  +1: markt
> +  +1: markt, remm
>    -1: 
>  
>  * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=45015
>    You can't use an unescaped quote if you quote the value with that character
>    http://svn.apache.org/viewvc?rev=657231&view=rev
>    +1: markt
> -  -1: 
> +  -1: remm (I think it is a good patch, but it throws an error for something which was working before,
> +            so I would think keeping it for the next release would be better)
How about making it optional so it only throws the error when the option is 
set or STRICT_SERVLET_COMPLIANCE is set?

Mark


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