--- jsr166/build.xml	2003/05/28 14:19:25	1.30
+++ jsr166/build.xml	2003/05/30 21:27:52	1.38
@@ -7,7 +7,7 @@
   Usage: ant [target]
 
   User-specific settings are read from user.properties.
-  See user.properties.sample for examples.
+  See user.properties.sample for an explanation of some useful settings.
 ------------------------------------------------------------------------------
   </description>
 
@@ -54,55 +54,11 @@
 
   <!-- Jar locations -->
   <property name="product.jar"          location="${build.lib.dir}/jsr166.jar"/>
-  <property name="javac.jar"            location="${lib.dir}/javac.jar"/>
-  <property name="collect.jar"          location="${lib.dir}/collect.jar"/>
   <property name="junit.jar"            location="${lib.dir}/junit.jar"/>
   <property name="rt.jar"               location="${java.home}/lib/rt.jar"/>
 
 
 
-  <!--
-   ! Bootclasspath munging for source compilation.
-   -->
-
-  <path id="javac.bootclasspath.prefix">
-    <!-- <pathelement location="${src.dir}"/> -->
-    <pathelement location="${javac.jar}"/>
-  </path>
-
-  <path id="javac.bootclasspath">
-    <!-- <pathelement location="${src.dir}"/> -->
-    <pathelement location="${collect.jar}"/>
-    <pathelement location="${rt.jar}"/>
-  </path>
-
-  <!-- Flatten bootclasspath prefix into a platform-appropriate string -->
-  <property name="javac.bootclasspath.prefix" refid="javac.bootclasspath.prefix"/>
-
-  <!-- Turn the flattened bootclasspath prefix into a javac argument -->
-  <property name="javac.args" value='-J-Xbootclasspath/p:${javac.bootclasspath.prefix}'/>
-
-
-
-  <!--
-   ! Bootclasspath munging for testing, so JUnit can test our local
-   ! modifications to java.*.
-   -->
-
-  <path id="test.classpath">
-    <pathelement location="${product.jar}"/>
-    <pathelement location="${build.testcases.dir}"/>
-    <pathelement location="${junit.jar}"/>
-  </path>
-
-  <!-- Flatten test classpath into a platform-appropriate string -->
-  <property name="test.classpath" refid="test.classpath"/>
-
-  <!-- Turn the flattened test classpath into a javac argument -->
-  <property name="test.javac.args" value='-Xbootclasspath/p:${test.classpath}'/>
-
-
-
   <!-- Files excluded from emulation and dist-docs -->
   <patternset id="emulation.excludes">
     <exclude name="java/util/Random.*"/>
@@ -114,12 +70,19 @@
   <!-- Main targets -->
 
   <target name="compile"
-          depends="init"
+          depends="init, configure-compiler, prepare-src"
           description="Compiles main sources to build folder">
 
+    <property name="prepare.src.dir" value="${src.dir}"/>
+
     <mkdir dir="${build.classes.dir}"/>
 
-    <javac srcdir="${src.dir}"
+<!--
+    <echo>javac ${gjc.args}</echo>
+    <echo>bootclasspath=${compile.bootclasspath}</echo>
+-->
+
+    <javac srcdir="${prepare.src.dir}"
           destdir="${build.classes.dir}"
             debug="${build.debug}"
        debuglevel="${build.debuglevel}"
@@ -127,8 +90,8 @@
            source="${build.sourcelevel}"
              fork="true">
 
-      <bootclasspath refid="javac.bootclasspath"/>
-      <compilerarg line="${javac.args} ${build.warnings.option}"/>
+      <compilerarg    line="${gjc.args}"/>
+      <bootclasspath refid="compile.bootclasspath"/>
 
     </javac>
 
@@ -136,12 +99,12 @@
 
 
   <target name="jar"
-          depends="check-emulation, init-jar, native-jar, emulation-jar"
+          depends="configure-emulation, init-jar, native-jar, emulation-jar"
           description="Builds library jar from compiled sources"/>
 
 
   <target name="test"
-          depends="init, check-junit, report-tests"
+          depends="init, configure-tests, report-tests"
           description="Runs all tests (requires JUnit 3.8.1 in ${ant.home}/lib)" />
 
 
@@ -200,16 +163,17 @@
   </target>
 
 
-  <!--
-   # javac -s doesn't reliably generate compilable code. It generates
-   # bridge methods (marked as "synthetic") that can have identical
-   # signatures to existing methods except for the return value.
-   -->
   <target name="strip"
-          depends="init">
+          depends="init, configure-compiler"
+          description="Strip generics from java source (not working yet)">
 
     <mkdir dir="${build.stripped.dir}"/>
 
+    <!--
+     # javac -s doesn't reliably generate compilable code. It generates
+     # bridge methods (marked as "synthetic") that can have identical
+     # signatures to existing methods except for the return value.
+     -->
     <javac srcdir="${src.dir}"
           destdir="${build.stripped.dir}"
             debug="${build.debug}"
@@ -218,8 +182,8 @@
            source="${build.sourcelevel}"
              fork="true">
 
-      <bootclasspath refid="javac.bootclasspath"/>
-      <compilerarg line="${javac.args} ${build.warnings.option} -s"/>
+      <compilerarg    line="${gjc.args} -s"/>
+      <bootclasspath refid="compile.bootclasspath"/>
 
     </javac>
 
@@ -271,22 +235,8 @@
 
   <!-- Internal targets -->
 
-  <target name="set-warnings-if" if="build.warnings">
-
-    <property name="build.warnings.option" value="-warnunchecked"/>
-
-  </target>
-
-
-  <target name="set-warnings-unless" unless="build.warnings">
-
-    <property name="build.warnings.option" value=""/>
 
-  </target>
-
-
-  <target name="init"
-          depends="set-warnings-if, set-warnings-unless">
+  <target name="init">
 
     <!-- Version is kept in a separate file -->
     <loadfile property="version" srcFile="version.properties"/>
@@ -314,7 +264,7 @@
 
 
   <target name="compile-emulation"
-          depends="init"
+          depends="init, configure-compiler"
           if="build.emulation.true">
 
     <mkdir dir="${build.emulation.dir}"/>
@@ -327,8 +277,8 @@
            source="${build.sourcelevel}"
              fork="true">
 
-      <bootclasspath refid="javac.bootclasspath"/>
-      <compilerarg line="${javac.args} ${build.warnings.option}"/>
+      <compilerarg    line="${gjc.args}"/>
+      <bootclasspath refid="compile.bootclasspath"/>
 
     </javac>
 
@@ -378,38 +328,83 @@
 
     <copy todir="${build.filter.src.dir}">
       <fileset dir="${src.dir}">
+        <include name="**/*.html"/>
+      </fileset>
+    </copy>
+
+    <!-- Not needed now, used for doccheck filtering:
+    <property name="generic.declarations"
+             value="public interface E {} public interface T {} public interface K {} public interface V {}"
+    />
+    -->
+
+    <copy todir="${build.filter.src.dir}">
+      <fileset dir="${src.dir}">
+        <exclude name="**/*.html"/>
         <patternset refid="emulation.excludes"/>
       </fileset>
       <filterchain>
+
+        <!--
+         # This filter gets rid of angle-bracketed type parameters
+         # so that javadoc can run on the result. The following
+         # heuristic seems to work:
+         #
+         # For all lines not starting with space(s)-asterisk-space(s),
+         #   replace <something> with a space, where there may be more
+         #   than one right angle bracket at the end, and "something"
+         #   must not contain parens or pipes. (This may need some
+         #   tweaking.)
+         -->
+
         <filterreader classname="jsr166.ant.filters.ReplaceFilter"
                       classpath="${build.ant.dir}">
-          <!--
-           # These arguments are to get rid of angle-bracketed type
-           # parameters so that javadoc can run on the result. The
-           # following heuristic that seems to work:
-           #
-           # For all lines not starting with space(s)-asterisk-space(s),
-           #   replace <something> with a space, where there may be more
-           #   than one right angle bracket at the end, and "something"
-           #   must not contain parens or pipes. (This may need some
-           #   tweaking.)
-           -->
           <param name="notmatching" value="^\s+\*\s.*$"/>
-          <param name="pattern" value="&lt;[^|>()]+?>+"/>
+          <param name="pattern"     value="&lt;[^|>()]+?>+"/>
           <param name="replacement" value=" "/>
         </filterreader>
+
+
+        <!--
+         # This filter uncomments lines beginning with "//@" so that
+         # javadoc can see imports that are needed to resolve links
+         # but that shouldn't be in the compiled code.
+         -->
+
         <filterreader classname="jsr166.ant.filters.ReplaceFilter"
                       classpath="${build.ant.dir}">
-          <!--
-           # These arguments are to uncomment lines beginning with
-           # "//@" so that javadoc can see imports that are needed
-           # to resolve links but that shouldn't be in the compiled
-           # code.
-           -->
-          <param name="matching" value="^//@.*$"/>
-          <param name="pattern" value="^//@"/>
+          <param name="matching"    value="^//@.*$"/>
+          <param name="pattern"     value="^//@"/>
           <param name="replacement" value=""/>
         </filterreader>
+
+
+        <!--
+         # The next two filters try to make the source look like
+         # something that doccheck can process. The first removes
+         # -source 1.4 assertions and the second adds in a bunch
+         # of single letter public nested marker interfaces so that
+         # the generic type parameters are recognized.
+         #
+         # Currently commented out because doccheck doesn't work. :-(
+         -->
+
+        <!--
+        <filterreader classname="jsr166.ant.filters.ReplaceFilter"
+                      classpath="${build.ant.dir}">
+          <param name="matching"    value="^\s*assert[\s ].*$"/>
+          <param name="pattern"     value="assert"/>
+          <param name="replacement" value="//assert"/>
+        </filterreader>
+
+        <filterreader classname="jsr166.ant.filters.ReplaceFilter"
+                      classpath="${build.ant.dir}">
+          <param name="matching"    value="^(.*(class|interface|implements) .*|)\{.*$"/>
+          <param name="pattern"     value="$"/>
+          <param name="replacement" value=" ${generic.declarations}"/>
+        </filterreader>
+        -->
+
       </filterchain>
     </copy>
 
@@ -421,6 +416,12 @@
 
     <mkdir dir="${build.testcases.dir}"/>
 
+<!--
+    <echo>javac ${gjc.args}</echo>
+    <echo>bootclasspath=${test.compile.bootclasspath}</echo>
+    <echo>classpath="${test.classpath}"</echo>
+-->
+
     <javac srcdir="${test.src.dir}"
           destdir="${build.testcases.dir}"
             debug="${build.debug}"
@@ -429,9 +430,9 @@
            source="${build.sourcelevel}"
              fork="true">
 
-      <bootclasspath refid="javac.bootclasspath"/>
-      <compilerarg line="${javac.args} ${build.warnings.option}"/>
-      <classpath refid="test.classpath"/>
+      <compilerarg    line="${gjc.args}"/>
+      <bootclasspath refid="test.compile.bootclasspath"/>
+      <classpath     refid="test.classpath"/>
 
     </javac>
 
@@ -453,7 +454,7 @@
                     dir="${build.reports.dir}"
                    fork="true">
 
-      <jvmarg value="${test.javac.args}"/>
+      <jvmarg value="-Xbootclasspath/p:${test.run.bootclasspath}"/>
 
       <formatter type="xml"/>
 
@@ -494,25 +495,140 @@
   </target>
 
 
-  <target name="check-junit">
+  <target name="configure-compiler">
 
-    <!-- junit.framework.Protectable is in JUnit 3.8.1 but not in 3.7 -->
-    <available property="junit.available"
-               classname="junit.framework.Protectable"/>
+    <property name="gjc.version"
+             value="2.0"/>
 
-    <fail message="Need JUnit 3.8.1 in ${ant.home}${file.separator}lib to run tests"
-          unless="junit.available"/>
+    <condition property="novariance.arg" value="-novariance">
+      <and>
+        <equals arg1="${gjc.version}" arg2="2.0"/>
+        <or>
+          <not><isset property="gjc.novariance"/></not>
+          <istrue value="${gjc.novariance}"/>
+        </or>
+      </and>
+    </condition>
+
+    <property name="novariance.arg"
+             value=""/>
+
+    <property name="gjc.dir"
+             value="${lib.dir}/gjc"/>
+
+    <property name="javac.jar"
+          location="${gjc.dir}/${gjc.version}/javac.jar"/>
+
+    <property name="collect.jar"
+          location="${gjc.dir}/${gjc.version}/collect${novariance.arg}.jar"/>
+
+
+    <condition property="warnunchecked.arg" value="-warnunchecked">
+      <istrue value="${gjc.warnunchecked}"/>
+    </condition>
+
+    <property name="warnunchecked.arg" value=""/>
+
+    <condition property="prepare.src.dir" value="${build.dir}/prepare-src">
+      <istrue value="${build.nothreads}"/>
+    </condition>
+
+
+    <!--
+     ! Bootclasspath munging for source compilation.
+     -->
+
+    <path id="pre.bootclasspath">
+      <pathelement location="${javac.jar}"/>
+    </path>
+
+    <path id="compile.bootclasspath">
+      <pathelement location="${collect.jar}"/>
+      <pathelement location="${rt.jar}"/>
+    </path>
+
+    <!-- Flatten paths into platform-appropriate strings -->
+    <property name="pre.bootclasspath"     refid="pre.bootclasspath"/>
+    <property name="compile.bootclasspath" refid="compile.bootclasspath"/>
+
+
+    <!-- Common options in javac invocations -->
+    <property name="gjc.args"
+             value="-J-Xbootclasspath/p:${pre.bootclasspath} ${warnunchecked.arg} ${novariance.arg}"
+    />
 
   </target>
 
 
-  <target name="check-emulation">
+  <target name="prepare-src"
+          depends="configure-compiler"
+          if="prepare.src.dir">
+
+    <mkdir dir="${prepare.src.dir}"/>
+    <copy todir="${prepare.src.dir}">
+      <fileset dir="${src.dir}">
+        <exclude name="java/lang/**"/>
+      </fileset>
+    </copy>
+
+  </target>
+
+
+  <target name="configure-emulation">
+
     <condition property="build.emulation.true">
       <or>
-        <isset property="build.emulation"/>
-        <os family="windows"/>
+        <and>
+          <os family="windows"/>
+          <not>
+            <isset property="build.emulation"/>
+          </not>
+        </and>
+        <istrue value="${build.emulation}"/>
       </or>
     </condition>
+
+  </target>
+
+
+  <target name="configure-tests"
+       depends="configure-compiler">
+
+    <!-- junit.framework.Protectable is in JUnit 3.8.1 but not in 3.7 -->
+    <available property="junit.available"
+               classname="junit.framework.Protectable"/>
+
+    <fail message="Need JUnit 3.8.1 in ${ant.home}${file.separator}lib to run tests"
+          unless="junit.available"/>
+
+
+    <!--
+     ! Bootclasspath munging for testing, so JUnit can test our local
+     ! modifications to java.*.
+     -->
+
+    <path id="test.classpath">
+      <pathelement location="${product.jar}"/>
+      <pathelement location="${build.testcases.dir}"/>
+      <pathelement location="${junit.jar}"/>
+    </path>
+
+    <path id="test.compile.bootclasspath">
+      <pathelement location="${javac.jar}"/>
+      <pathelement location="${collect.jar}"/>
+      <pathelement location="${rt.jar}"/>
+    </path>
+
+    <path id="test.run.bootclasspath">
+      <pathelement location="${javac.jar}"/>
+      <path refid="test.classpath"/>
+    </path>
+
+    <!-- Flatten test classpaths into platform-appropriate strings -->
+    <property name="test.classpath"             refid="test.classpath"/>
+    <property name="test.compile.bootclasspath" refid="test.compile.bootclasspath"/>
+    <property name="test.run.bootclasspath"     refid="test.run.bootclasspath"/>
+
   </target>