You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by bi...@apache.org on 2008/09/25 02:36:26 UTC
svn commit: r698785 - /tomcat/current/tc5.5.x/STATUS.txt
Author: billbarker
Date: Wed Sep 24 17:36:26 2008
New Revision: 698785
URL: http://svn.apache.org/viewvc?rev=698785&view=rev
Log:
Vote
Modified:
tomcat/current/tc5.5.x/STATUS.txt
Modified: tomcat/current/tc5.5.x/STATUS.txt
URL: http://svn.apache.org/viewvc/tomcat/current/tc5.5.x/STATUS.txt?rev=698785&r1=698784&r2=698785&view=diff
==============================================================================
--- tomcat/current/tc5.5.x/STATUS.txt (original)
+++ tomcat/current/tc5.5.x/STATUS.txt Wed Sep 24 17:36:26 2008
@@ -116,4 +116,7 @@
Code clean-up
https://issues.apache.org/bugzilla/attachment.cgi?id=22596
+1: markt
- -1:
+ -1: billbarker - The patch is bad in that it will break SSL support. The patch only checks the running version of
+ the JVM, not the version that was used to compile it. So the patch won't allow running Tomcat on
+ a 1.5 or 1.6 JVM if it was compiled using 1.4. A better patch would just remove the 13Factory,
+ since it only applies to TC 3.x & 4.x which aren't likely to release again.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org
Re: svn commit: r698785 - /tomcat/current/tc5.5.x/STATUS.txt
Posted by Mark Thomas <ma...@apache.org>.
billbarker@apache.org wrote:
> Author: billbarker
> Date: Wed Sep 24 17:36:26 2008
> New Revision: 698785
>
> URL: http://svn.apache.org/viewvc?rev=698785&view=rev
> Log:
> Vote
>
> Modified:
> tomcat/current/tc5.5.x/STATUS.txt
>
> Modified: tomcat/current/tc5.5.x/STATUS.txt
> URL: http://svn.apache.org/viewvc/tomcat/current/tc5.5.x/STATUS.txt?rev=698785&r1=698784&r2=698785&view=diff
> ==============================================================================
> --- tomcat/current/tc5.5.x/STATUS.txt (original)
> +++ tomcat/current/tc5.5.x/STATUS.txt Wed Sep 24 17:36:26 2008
> @@ -116,4 +116,7 @@
> Code clean-up
> https://issues.apache.org/bugzilla/attachment.cgi?id=22596
> +1: markt
> - -1:
> + -1: billbarker - The patch is bad in that it will break SSL support. The patch only checks the running version of
> + the JVM, not the version that was used to compile it. So the patch won't allow running Tomcat on
> + a 1.5 or 1.6 JVM if it was compiled using 1.4. A better patch would just remove the 13Factory,
> + since it only applies to TC 3.x & 4.x which aren't likely to release again.
I am about to do a 4.1.x release for the recent security fixes so I'll just
reject this patch. Future 4.1.x releases will depend on if we need to fix
any security issues. That said, no-one seems to be clamouring for a 4.1.x
release.
Cheers,
Mark
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org