Author: markt
Date: Wed Apr  9 15:41:28 2014
New Revision: 1585997

URL: http://svn.apache.org/r1585997
Log:
Fix http://issues.apache.org/bugzilla/show_bug.cgi?id=56283
Add support for running Tomcat 6 with ecj-P20140317-1600.jar (as drop-in 
replacement for ecj-4.3.1.jar).
Add support for value "1.8" for the compilerSourceVM and compilerTargetVM 
options.
Note that ecj-P20140317-1600.jar can only be used when running with Java 6 or 
later. The "1.8" options make sense only when running with Java 8 (or later).

Modified:
    tomcat/tc6.0.x/trunk/STATUS.txt
    tomcat/tc6.0.x/trunk/java/org/apache/catalina/startup/TldConfig.java
    tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/JDTCompiler.java
    tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/TldLocationsCache.java
    tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=1585997&r1=1585996&r2=1585997&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Wed Apr  9 15:41:28 2014
@@ -28,17 +28,6 @@ None
 PATCHES PROPOSED TO BACKPORT:
   [ New proposals should be added at the end of the list ]
 
-* Fix http://issues.apache.org/bugzilla/show_bug.cgi?id=56283
-  Add support for running Tomcat 6 with ecj-P20140317-1600.jar (as drop-in
-  replacement for ecj-4.3.1.jar). Add support for value "1.8" for the
-  compilerSourceVM and compilerTargetVM options.
-  Note that, per previous comments, ecj-P20140317-1600.jar can only be used 
when
-  running with Java 6 or later. The "1.8" options make sense only when running
-  with Java 8 (or later).
-  https://issues.apache.org/bugzilla/attachment.cgi?id=31476&action=diff
-  +1: markt, schultz, kkolinko
-  -1:
-
 * Add a work around for validating XML documents (often TLDs) that use
   just the file name to refer to refer to the JavaEE schema on which they
   are based.

Modified: tomcat/tc6.0.x/trunk/java/org/apache/catalina/startup/TldConfig.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/catalina/startup/TldConfig.java?rev=1585997&r1=1585996&r2=1585997&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/java/org/apache/catalina/startup/TldConfig.java 
(original)
+++ tomcat/tc6.0.x/trunk/java/org/apache/catalina/startup/TldConfig.java Wed 
Apr  9 15:41:28 2014
@@ -94,6 +94,8 @@ public final class TldConfig  implements
         noTldJars.add("ecj-4.2.1.jar");
         noTldJars.add("ecj-4.2.2.jar");
         noTldJars.add("ecj-4.3.1.jar");
+        noTldJars.add("ecj-4.3.2.jar");
+        noTldJars.add("ecj-P20140317-1600.jar");
         noTldJars.add("jsp-api.jar");
         noTldJars.add("servlet-api.jar");
         noTldJars.add("tomcat-coyote.jar");

Modified: tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/JDTCompiler.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/JDTCompiler.java?rev=1585997&r1=1585996&r2=1585997&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/JDTCompiler.java 
(original)
+++ tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/JDTCompiler.java Wed 
Apr  9 15:41:28 2014
@@ -310,6 +310,9 @@ public class JDTCompiler extends org.apa
             } else if(opt.equals("1.7")) {
                 settings.put(CompilerOptions.OPTION_Source,
                              CompilerOptions.VERSION_1_7);
+            } else if(opt.equals("1.8")) {
+                settings.put(CompilerOptions.OPTION_Source,
+                             "1.8"); // CompilerOptions.VERSION_1_8
             } else {
                 log.warn("Unknown source VM " + opt + " ignored.");
                 settings.put(CompilerOptions.OPTION_Source,
@@ -351,6 +354,11 @@ public class JDTCompiler extends org.apa
                              CompilerOptions.VERSION_1_7);
                 settings.put(CompilerOptions.OPTION_Compliance,
                         CompilerOptions.VERSION_1_7);
+            } else if(opt.equals("1.8")) {
+                settings.put(CompilerOptions.OPTION_TargetPlatform,
+                        "1.8"); // CompilerOptions.VERSION_1_8
+                settings.put(CompilerOptions.OPTION_Compliance,
+                        "1.8"); // CompilerOptions.VERSION_1_8
             } else {
                 log.warn("Unknown target VM " + opt + " ignored.");
                 settings.put(CompilerOptions.OPTION_TargetPlatform,

Modified: 
tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/TldLocationsCache.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/TldLocationsCache.java?rev=1585997&r1=1585996&r2=1585997&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/TldLocationsCache.java 
(original)
+++ tomcat/tc6.0.x/trunk/java/org/apache/jasper/compiler/TldLocationsCache.java 
Wed Apr  9 15:41:28 2014
@@ -134,6 +134,8 @@ public class TldLocationsCache {
         noTldJars.add("ecj-4.2.1.jar");
         noTldJars.add("ecj-4.2.2.jar");
         noTldJars.add("ecj-4.3.1.jar");
+        noTldJars.add("ecj-4.3.2.jar");
+        noTldJars.add("ecj-P20140317-1600.jar");
         noTldJars.add("jsp-api.jar");
         noTldJars.add("servlet-api.jar");
         noTldJars.add("tomcat-coyote.jar");

Modified: tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml?rev=1585997&r1=1585996&r2=1585997&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml Wed Apr  9 15:41:28 2014
@@ -122,6 +122,14 @@
         <bug>56265</bug>: Do not escape values of dynamic tag attributes
         containing EL expressions. (kkolinko)
       </fix>
+      <fix>
+        <bug>56283</bug>: Add support for running Tomcat 6 with
+        ecj-P20140317-1600.jar (as drop-in replacement for ecj-4.3.1.jar). Add
+        support for value "1.8" for the <code>compilerSourceVM</code> and
+        <code>compilerTargetVM</code> options. Note that ecj-P20140317-1600.jar
+        can only be used when running with Java 6 or later. The "1.8" options
+        make sense only when running with Java 8 (or later). (kkolinko)
+      </fix>
     </changelog>
   </subsection>
   <subsection name="Web applications">



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

Reply via email to