Author: markt
Date: Mon Oct 31 20:11:12 2011
New Revision: 1195649

URL: http://svn.apache.org/viewvc?rev=1195649&view=rev
Log:
Update Eclipse compiler to 3.7 and switch to using ecj.jar

Added:
    tomcat/tc5.5.x/trunk/build/resources/License.rtf
      - copied, changed from r1195441, tomcat/tc6.0.x/trunk/res/License.rtf
Modified:
    tomcat/tc5.5.x/trunk/LICENSE
    tomcat/tc5.5.x/trunk/STATUS.txt
    tomcat/tc5.5.x/trunk/build/RELEASE-NOTES
    tomcat/tc5.5.x/trunk/build/build.properties.default
    tomcat/tc5.5.x/trunk/build/build.xml
    
tomcat/tc5.5.x/trunk/container/catalina/src/share/org/apache/catalina/startup/TldConfig.java
    tomcat/tc5.5.x/trunk/container/webapps/docs/changelog.xml
    tomcat/tc5.5.x/trunk/container/webapps/docs/class-loader-howto.xml
    tomcat/tc5.5.x/trunk/container/webapps/docs/jasper-howto.xml
    tomcat/tc5.5.x/trunk/eclipse.classpath
    tomcat/tc5.5.x/trunk/jasper/build.xml

Modified: tomcat/tc5.5.x/trunk/LICENSE
URL: 
http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/LICENSE?rev=1195649&r1=1195648&r2=1195649&view=diff
==============================================================================
--- tomcat/tc5.5.x/trunk/LICENSE (original)
+++ tomcat/tc5.5.x/trunk/LICENSE Mon Oct 31 20:11:12 2011
@@ -263,7 +263,7 @@ For the jmx.jar component:
  */
 
 
-For the jasper-jdt.jar component:
+For the Eclipse JDT Java compiler:
 
 Eclipse Public License - v 1.0
 

Modified: tomcat/tc5.5.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/STATUS.txt?rev=1195649&r1=1195648&r2=1195649&view=diff
==============================================================================
--- tomcat/tc5.5.x/trunk/STATUS.txt (original)
+++ tomcat/tc5.5.x/trunk/STATUS.txt Mon Oct 31 20:11:12 2011
@@ -27,23 +27,6 @@ PATCHES ACCEPTED TO BACKPORT FROM TRUNK/
 PATCHES PROPOSED TO BACKPORT:
   [ New proposals should be added at the end of the list ]
 
-* Update Eclipse compiler and switch to using ecj.jar
-  http://people.apache.org/~markt/patches/2011-08-25-lib-update-tc5-v2.patch
-  +1: markt
-  +1: kkolinko:
-        + update build/resources/License.rtf to match LICENSE file
-        + in class-loader-howto.xml sort the JARs list alphabetically
-        I've run jarcheck tool against the archive. Results are in
-         http://people.apache.org/~kkolinko/patches/jarcheck/
-        Most classes are JDK 1.2 compatible. Some are JDK 1.6 (esp. 
annotations processor),
-        but it is the same in ecj 3.3.1 that is currently used.
-        Those classes are absent in jasper-compiler-jdt.jar that we would
-        generate, so their presence does not matter.
-  +1: rjung:
-        I suggest we switch after releasing 5.5.34 so that the change isn't 
introduced
-        immediately before tagging.
-  -1:
-
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=52059
   http://svn.apache.org/viewvc?rev=1186763&view=rev
   Fix a typo in Windows uninstaller:

Modified: tomcat/tc5.5.x/trunk/build/RELEASE-NOTES
URL: 
http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/build/RELEASE-NOTES?rev=1195649&r1=1195648&r2=1195649&view=diff
==============================================================================
--- tomcat/tc5.5.x/trunk/build/RELEASE-NOTES (original)
+++ tomcat/tc5.5.x/trunk/build/RELEASE-NOTES Mon Oct 31 20:11:12 2011
@@ -86,7 +86,7 @@ for use by web applications (by placing 
 * commons-el.jar (Commons Expression Language 1.0)
 * commons-logging-api.jar (Commons Logging API 1.0.x)
 * jasper-compiler.jar (Jasper 2 Compiler)
-* jasper-compiler-jdt.jar (Eclipse JDT Java compiler)
+* ecj-x.y.z.jar (Eclipse JDT Java compiler)
 * jasper-runtime.jar (Jasper 2 Runtime)
 * jsp-api.jar (JSP 2.0 API)
 * naming-common.jar (JNDI Context implementation)

Modified: tomcat/tc5.5.x/trunk/build/build.properties.default
URL: 
http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/build/build.properties.default?rev=1195649&r1=1195648&r2=1195649&view=diff
==============================================================================
--- tomcat/tc5.5.x/trunk/build/build.properties.default (original)
+++ tomcat/tc5.5.x/trunk/build/build.properties.default Mon Oct 31 20:11:12 2011
@@ -146,10 +146,14 @@ xerces.loc=${base-xerces.loc}/j/binaries
 
 
 # ----- Eclipse JDT, version 3.3.1 or later -----
-jdt.home=${base.path}/eclipse/plugins
-jdt.lib=${jdt.home}
-jdt.jar=${jdt.lib}/org.eclipse.jdt.core_3.3.1.v_780_R33x.jar
-jdt.loc=http://archive.eclipse.org/eclipse/downloads/drops/R-3.3.1-200709211145/eclipse-JDT-3.3.1.zip
+jdt.version=3.7
+jdt.release=R-3.7-201106131736
+jdt.home=${base.path}/ecj-${jdt.version}
+jdt.jar=${jdt.home}/ecj-${jdt.version}.jar
+# The download will be moved to the archive area eventually. We are taking 
care of that in advance.
+# Note older JARs were called ecj.jar. Newer JARs are called 
ecj-${jdt.version}.jar
+jdt.loc.1=http://archive.eclipse.org/eclipse/downloads/drops/${jdt.release}/ecj-${jdt.version}.jar
+jdt.loc.2=http://download.eclipse.org/eclipse/downloads/drops/${jdt.release}/ecj-${jdt.version}.jar
 
 # ----- Tomcat native library -----
 tomcat-native.version=1.1.22

Modified: tomcat/tc5.5.x/trunk/build/build.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/build/build.xml?rev=1195649&r1=1195648&r2=1195649&view=diff
==============================================================================
--- tomcat/tc5.5.x/trunk/build/build.xml (original)
+++ tomcat/tc5.5.x/trunk/build/build.xml Mon Oct 31 20:11:12 2011
@@ -87,9 +87,6 @@
   <property name="tomcat-dbcp.home" value="${base.path}/tomcat-deps/dbcp" />
   <property name="tomcat-dbcp.jar"
        value="${tomcat-dbcp.home}/naming-factory-dbcp.jar"/>
-  <property name="jasper-compiler-jdt.home" 
value="${base.path}/tomcat-deps/jdt" />
-  <property name="jasper-compiler-jdt.jar"
-       value="${jasper-compiler-jdt.home}/jasper-compiler-jdt.jar"/>
 
   <!-- Some compilers will disable debugging if true. And it doesn't do 
anything
        in most cases -->
@@ -234,7 +231,7 @@
 
     <!-- <copy todir="${tomcat.build}/common/lib" file="${ant.jar}"/>
     <copy todir="${tomcat.build}/common/lib" file="${ant-launcher.jar}"/> -->
-    <copy todir="${tomcat.build}/common/lib" 
file="${jasper-compiler-jdt.jar}"/>
+    <copy todir="${tomcat.build}/common/lib" file="${jdt.jar}"/>
   </target>
 
   <!-- ====================== Build all components =================== -->
@@ -706,7 +703,7 @@
 
 
   <target name="build-depends" depends="init"
-          description="Builds various dependent components - APIs, dbcp, jdt">
+          description="Builds various dependent components - APIs, dbcp">
     <antcall target="build-servletapi"/>
     <antcall target="build-jspapi"/>
 
@@ -726,12 +723,12 @@
 
     <antcall target="build-tomcat-dbcp" />
 
-    <antcall target="downloadzip">
-      <param name="sourcefile" value="${jdt.loc}"/>
+    <antcall target="downloadfile-2">
+      <param name="sourcefile.1" value="${jdt.loc.1}"/>
+      <param name="sourcefile.2" value="${jdt.loc.2}"/>
       <param name="destfile" value="${jdt.jar}"/>
-      <param name="destdir" value="${base.path}"/>
+      <param name="destdir" value="${jdt.home}"/>
     </antcall>
-    <antcall target="build-jasper-compiler-jdt" />
   </target>
 
   <target name="build-tomcat-dbcp">
@@ -795,24 +792,6 @@
     </jar>
   </target>
 
-  <target name="build-jasper-compiler-jdt">
-       <mkdir dir="${jasper-compiler-jdt.home}"/>
-       <antcall target="-build-jasper-compiler-jdt">
-          <param name="basedir" value="${jasper-compiler-jdt.home}" />
-       </antcall>
-      </target>
-
-      <target name="-build-jasper-compiler-jdt">
-        <unjar src="${jdt.jar}" dest="${jasper-compiler-jdt.home}" />
-        <jar destfile="${jasper-compiler-jdt.jar}">
-            <fileset dir="${jasper-compiler-jdt.home}">
-                <include name="org/eclipse/jdt/core/compiler/**"/>
-                <include name="org/eclipse/jdt/internal/compiler/**"/>
-                <include 
name="org/eclipse/jdt/internal/core/util/CommentRecorder*"/>
-            </fileset>
-        </jar>
-  </target>
-
   <target name="build-webapps" depends="init"
           description="build  webapps">
 
@@ -1009,7 +988,7 @@
     <copy todir="${tomcat.embed}/lib">
       <fileset dir="${tomcat.build}/common/lib">
         <include name="jasper-compiler.jar"/>
-        <include name="jasper-compiler-jdt.jar"/>
+        <include name="ecj-${jdt.version}.jar"/>
       </fileset>
     </copy>
 
@@ -1162,7 +1141,6 @@
   <target name="clean-depend"
    description="Clean depend src components">
     <delete dir="${tomcat-dbcp.home}"/>    
-    <delete dir="${jasper-compiler-jdt.home}"/>
   </target>
   
   <target name="clean"
@@ -2174,6 +2152,38 @@
     <get src="${sourcefile}" dest="${destfile}" />
   </target>
 
+  <target name="downloadfile-2" unless="exist" depends="testexist">
+    <!-- Download the file from the two alternative locations -->
+    <mkdir dir="${destdir}" />
+
+    <antcall target="trydownload">
+      <param name="sourcefile" value="${sourcefile.1}" />
+    </antcall>
+
+    <antcall target="trydownload">
+      <param name="sourcefile" value="${sourcefile.2}" />
+    </antcall>
+
+    <available file="${destfile}" property="exist"/>
+    <fail unless="exist" message="Failed to download [${destfile}]. All 
download sources are unavailable." />
+  </target>
+
+  <target name="trydownload.check" depends="setproxy">
+    <condition property="trydownload.run">
+      <and>
+        <not>
+          <available file="${destfile}" />
+        </not>
+        <http url="${sourcefile}" />
+      </and>
+    </condition>
+  </target>
+
+  <target name="trydownload" if="trydownload.run" depends="trydownload.check">
+    <!-- Downloads a file if not yet downloaded and the source URL is 
available -->
+    <get src="${sourcefile}" dest="${destfile}" />
+  </target>
+
 
   <!-- ==================== Modeler tricks ====================
   -->

Copied: tomcat/tc5.5.x/trunk/build/resources/License.rtf (from r1195441, 
tomcat/tc6.0.x/trunk/res/License.rtf)
URL: 
http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/build/resources/License.rtf?p2=tomcat/tc5.5.x/trunk/build/resources/License.rtf&p1=tomcat/tc6.0.x/trunk/res/License.rtf&r1=1195441&r2=1195649&rev=1195649&view=diff
==============================================================================
Binary files - no diff available.

Modified: 
tomcat/tc5.5.x/trunk/container/catalina/src/share/org/apache/catalina/startup/TldConfig.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/container/catalina/src/share/org/apache/catalina/startup/TldConfig.java?rev=1195649&r1=1195648&r2=1195649&view=diff
==============================================================================
--- 
tomcat/tc5.5.x/trunk/container/catalina/src/share/org/apache/catalina/startup/TldConfig.java
 (original)
+++ 
tomcat/tc5.5.x/trunk/container/catalina/src/share/org/apache/catalina/startup/TldConfig.java
 Mon Oct 31 20:11:12 2011
@@ -88,7 +88,7 @@ public final class TldConfig implements 
         noTldJars.add("commons-logging-api.jar");
         noTldJars.add("commons-modeler.jar");
         noTldJars.add("jasper-compiler.jar");
-        noTldJars.add("jasper-compiler-jdt.jar");
+        noTldJars.add("ecj-3.7.jar");
         noTldJars.add("jasper-runtime.jar");
         noTldJars.add("jsp-api.jar");
         noTldJars.add("naming-resources.jar");

Modified: tomcat/tc5.5.x/trunk/container/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/container/webapps/docs/changelog.xml?rev=1195649&r1=1195648&r2=1195649&view=diff
==============================================================================
--- tomcat/tc5.5.x/trunk/container/webapps/docs/changelog.xml (original)
+++ tomcat/tc5.5.x/trunk/container/webapps/docs/changelog.xml Mon Oct 31 
20:11:12 2011
@@ -40,6 +40,13 @@
  General, Catalina, Coyote, Jasper, Cluster, Webapps
 -->
 <section name="Tomcat 5.5.35 (jim)" rtext="">
+  <subsection name="General">
+    <changelog>
+      <update>
+        Update Eclipse compiler to 3.7 and switch to using ecj.jar. (markt)    
    
+      </update>
+    </changelog>
+  </subsection>
   <subsection name="Webapps">
     <changelog>
       <fix>

Modified: tomcat/tc5.5.x/trunk/container/webapps/docs/class-loader-howto.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/container/webapps/docs/class-loader-howto.xml?rev=1195649&r1=1195648&r2=1195649&view=diff
==============================================================================
--- tomcat/tc5.5.x/trunk/container/webapps/docs/class-loader-howto.xml 
(original)
+++ tomcat/tc5.5.x/trunk/container/webapps/docs/class-loader-howto.xml Mon Oct 
31 20:11:12 2011
@@ -139,8 +139,8 @@ loaders as it is initialized:</p>
     <ul>
     <li><em>commons-el.jar</em> - Jakarta commons el, implementing the 
         expression language used by Jasper.</li>
+    <li><em>ecj-x.y.x.jar</em> - The Eclipse JDT Java compiler.</li>
     <li><em>jasper-compiler.jar</em> - The JSP 2.0 compiler.</li>
-    <li><em>jasper-compiler-jdt.jar</em> - The Eclipse JDT Java compiler.</li>
     <li><em>jasper-runtime.jar</em> - The JSP 2.0 runtime.</li>
     <li><em>jsp-api.jar</em> - The JSP 2.0 API.</li>
     <li><em>naming-common.jar</em> - The JNDI implementation used by Tomcat 5

Modified: tomcat/tc5.5.x/trunk/container/webapps/docs/jasper-howto.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/container/webapps/docs/jasper-howto.xml?rev=1195649&r1=1195648&r2=1195649&view=diff
==============================================================================
--- tomcat/tc5.5.x/trunk/container/webapps/docs/jasper-howto.xml (original)
+++ tomcat/tc5.5.x/trunk/container/webapps/docs/jasper-howto.xml Mon Oct 31 
20:11:12 2011
@@ -168,7 +168,7 @@ pages. This new compiler will be updated
 possible.</p>
 
 <p>Apache Ant, which was used in previous Tomcat releases, can be used instead 
instead of 
-the new compiler by simply removing the 
<code>common/lib/jasper-compiler-jdt.jar</code> file, 
+the new compiler by simply removing the <code>common/lib/ecj-x.y.z.jar</code> 
file, 
 and placing the <code>ant.jar</code> file from the latest Ant distribution in 
the 
 <code>common/lib</code> folder.  If you do this, you also need to use the 
"javac"
 argument to catalina.sh.</p>

Modified: tomcat/tc5.5.x/trunk/eclipse.classpath
URL: 
http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/eclipse.classpath?rev=1195649&r1=1195648&r2=1195649&view=diff
==============================================================================
--- tomcat/tc5.5.x/trunk/eclipse.classpath (original)
+++ tomcat/tc5.5.x/trunk/eclipse.classpath Mon Oct 31 20:11:12 2011
@@ -56,7 +56,7 @@
        <classpathentry kind="var" 
path="TOMCAT_LIBS_BASE/javamail-1.4.1/mail.jar"/>
        <classpathentry kind="var" 
path="TOMCAT_LIBS_BASE/jaf-1.1.1/activation.jar"/>
        <classpathentry kind="var" 
path="TOMCAT_LIBS_BASE/tomcat-deps/dbcp/naming-factory-dbcp.jar"/>
-       <classpathentry kind="var" 
path="TOMCAT_LIBS_BASE/tomcat-deps/jdt/jasper-compiler-jdt.jar"/>
+       <classpathentry kind="var" path="TOMCAT_LIBS_BASE/ecj-3.7/ecj-3.7.jar"/>
        <classpathentry kind="var" 
path="TOMCAT_LIBS_BASE/commons-el-1.0/commons-el.jar"/>
        <classpathentry kind="con" 
path="org.eclipse.jdt.junit.JUNIT_CONTAINER/3"/>
        <classpathentry kind="var" 
path="TOMCAT_LIBS_BASE/commons-launcher/bin/commons-launcher.jar"/>

Modified: tomcat/tc5.5.x/trunk/jasper/build.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/jasper/build.xml?rev=1195649&r1=1195648&r2=1195649&view=diff
==============================================================================
--- tomcat/tc5.5.x/trunk/jasper/build.xml (original)
+++ tomcat/tc5.5.x/trunk/jasper/build.xml Mon Oct 31 20:11:12 2011
@@ -46,7 +46,7 @@
     <pathelement location="${servlet-api.jar}"/>
     <pathelement location="${jsp-api.jar}"/>
     <pathelement location="${tools.jar}"/>
-    <pathelement location="${jasper-compiler-jdt.jar}"/>
+    <pathelement location="${jdt.jar}"/>
     <pathelement location="${xerces.jar}"/>
     <pathelement location="${xercesImpl.jar}"/>
     <pathelement location="${xml-apis.jar}"/>
@@ -284,7 +284,7 @@
           tofile="${jasper.deploy}/common/lib/jasper-runtime.jar" />
 
     <!-- Copy JARs -->
-    <copy todir="${jasper.deploy}/common/lib" 
file="${jasper-compiler-jdt.jar}"/>
+    <copy todir="${jasper.deploy}/common/lib" file="${jdt.jar}"/>
     <copy todir="${jasper.deploy}/common/lib" file="${jsp-api.jar}"/>
     <copy todir="${jasper.deploy}/common/lib" file="${commons-el.jar}"/>
 



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

Reply via email to