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 2014/11/09 12:23:40 UTC

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

Author: kkolinko
Date: Sun Nov  9 11:23:39 2014
New Revision: 1637656

URL: http://svn.apache.org/r1637656
Log:
Proposed patch does not compile.

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=1637656&r1=1637655&r2=1637656&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Sun Nov  9 11:23:39 2014
@@ -54,7 +54,16 @@ PATCHES PROPOSED TO BACKPORT:
   align the Tomcat 6 code with the code in Tomcat 7.
   http://people.apache.org/~markt/patches/2014-11-06-bug56780-tc6-v2.patch
   +1: markt, remm
-  -1:
+  -1: kkolinko: It does not compile with Java 5.
+
+       First, Java 5 does not allow @Override annotations when implementing
+       method declared by an interface.
+
+       Second, once I remove those overrides, compilation fails at
+       "context.getSupportedSSLParameters().getCipherSuites();" line
+       in JSSESocketFactory.java line 539. The method
+       SSLContext.getSupportedSSLParameters() is @since Java 6.
+       https://docs.oracle.com/javase/6/docs/api/javax/net/ssl/SSLContext.html
 
 
 PATCHES/ISSUES THAT ARE STALLED:



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