--- jsr166/build.xml	2013/01/20 07:43:30	1.117
+++ jsr166/build.xml	2013/01/21 22:03:09	1.121
@@ -62,7 +62,7 @@
   <property name="build.jsr166e.classes.dir"    location="${build.jsr166e.dir}/classes"/>
   <property name="build.extra166y.classes.dir"  location="${build.extra166y.dir}/classes"/>
 
-  <!-- JDK binary locations -->
+  <!-- JDK locations -->
   <property name="jdks.home"  location="${user.home}/jdk"/>
 
   <macrodef name="defjdklocations">
@@ -72,10 +72,17 @@
     <property name="java@{v}"       location="${jdk@{v}.home}/bin/java"/>
     <property name="javac@{v}"      location="${jdk@{v}.home}/bin/javac"/>
     <property name="javadoc@{v}"    location="${jdk@{v}.home}/bin/javadoc"/>
-    <property name="bootdir@{v}"    location="${jdk@{v}.home}/jre/lib"/>
     <property name="jdk@{v}src.dir" location="${jdks.home}/src/jdk@{v}/jdk/src/share/classes"/>
-    <property name="bootclasspath@{v}"
-     value="${bootdir@{v}}/resources.jar:${bootdir@{v}}/rt.jar:${bootdir@{v}}/jsse.jar:${bootdir@{v}}/jce.jar:${bootdir@{v}}/:${bootdir@{v}}/charsets.jar"/>
+    <local name="boot.jar.dir"/>
+    <property name="boot.jar.dir"   location="${jdk@{v}.home}/jre/lib"/>
+    <path id="bootclasspath@{v}">
+      <pathelement path="${boot.jar.dir}/resources.jar"/>
+      <pathelement path="${boot.jar.dir}/rt.jar"/>
+      <pathelement path="${boot.jar.dir}/jsse.jar"/>
+      <pathelement path="${boot.jar.dir}/jce.jar"/>
+      <pathelement path="${boot.jar.dir}/charsets.jar"/>
+    </path>
+    <property name="bootclasspath@{v}" value="${toString:bootclasspath@{v}}"/>
     </sequential>
   </macrodef>
 
@@ -83,49 +90,6 @@
   <defjdklocations v="7"/>
   <defjdklocations v="8"/>
 
-  <macrodef name="run-tck-tests">
-    <attribute name="target"/>
-    <attribute name="workdir"/>
-    <attribute name="product.jar" default="${product.jar}"/>
-    <attribute name="jvmflags" default=""/>
-    <sequential>
-
-    <mkdir dir="@{workdir}/tck-classes"/>
-
-    <javac srcdir="${tck.src.dir}"
-           destdir="@{workdir}/tck-classes"
-           debug="${build.debug}"
-           debuglevel="${build.debuglevel}"
-           deprecation="${build.deprecation}"
-           source="6"
-           classpath="${junit.jar}"
-           bootclasspath="@{product.jar}:${bootclasspath6}"
-           includeAntRuntime="false"
-           includeJavaRuntime="false"
-           executable="${javac@{target}}"
-           fork="true">
-
-      <include name="**/*.java"/>
-      <compilerarg value="-XDignore.symbol.file=true"/>
-      <compilerarg value="-Xlint:all,-unchecked,-rawtypes,-serial,-deprecation"/>
-      <compilerarg line="${build.args}"/>
-    </javac>
-
-    <java classname="JSR166TestCase"
-          failonerror="true"
-          jvm="${java@{target}}"
-          fork="true">
-	<jvmarg value="-Xbootclasspath/p:@{product.jar}"/>
-	<jvmarg line="@{jvmflags}"/>
-	<classpath>
-	  <pathelement location="${junit.jar}"/>
-	  <pathelement location="@{workdir}/tck-classes"/>
-	</classpath>
-    </java>
-
-    </sequential>
-  </macrodef>
-
   <!-- Source locations -->
   <property name="src.dir"              location="${basedir}/src/main"/>
   <property name="test.src.dir"         location="${basedir}/src/test"/>
@@ -158,9 +122,6 @@
   <property name="extra166y.jar"    location="${build.extra166y.dir}/extra166y.jar"/>
   <property name="junit.jar"        location="${lib.dir}/junit.jar"/>
 
-  <!-- Bootclasspath argument -->
-  <property name="bootclasspath.args"   value="-Xbootclasspath/p:${product.jar}"/>
-
   <!-- Canonical location of jdk API docs, to use with javadoc link attribute -->
   <property name="jdkapi5docs.url"      value="http://docs.oracle.com/javase/1.5.0/docs/api/"/>
   <property name="jdkapi6docs.url"      value="http://docs.oracle.com/javase/6/docs/api/"/>
@@ -184,16 +145,57 @@
     <pathelement location="${junit.jar}"/>
   </path>
 
+  <macrodef name="run-tck-tests">
+    <attribute name="target"/>
+    <attribute name="workdir"/>
+    <attribute name="product.jar" default="${product.jar}"/>
+    <attribute name="jvmflags" default=""/>
+    <sequential>
+
+    <mkdir dir="@{workdir}/tck-classes"/>
+
+    <javac srcdir="${tck.src.dir}"
+           destdir="@{workdir}/tck-classes"
+           debug="${build.debug}"
+           debuglevel="${build.debuglevel}"
+           deprecation="${build.deprecation}"
+           source="6"
+           classpath="${junit.jar}"
+           bootclasspath="@{product.jar}:${bootclasspath6}"
+           includeAntRuntime="false"
+           includeJavaRuntime="false"
+           executable="${javac@{target}}"
+           fork="true">
+
+      <include name="*.java"/>
+      <compilerarg value="-XDignore.symbol.file=true"/>
+      <compilerarg value="-Xlint:all,-unchecked,-rawtypes,-serial,-deprecation"/>
+      <compilerarg line="${build.args}"/>
+
+    </javac>
+
+    <java classname="JSR166TestCase"
+          failonerror="true"
+          jvm="${java@{target}}"
+          fork="true">
+	<jvmarg value="-Xbootclasspath/p:@{product.jar}"/>
+	<jvmarg line="@{jvmflags}"/>
+	<classpath>
+	  <pathelement location="${junit.jar}"/>
+	  <pathelement location="@{workdir}/tck-classes"/>
+	</classpath>
+    </java>
+
+    </sequential>
+  </macrodef>
+
   <!-- ALoops classpath -->
   <path id="loops.classpath">
     <pathelement location="${build.loops.dir}"/>
   </path>
 
   <!-- Support @jls tag, used in jdk8+ javadoc -->
-  <!-- TODO: How do we get &trade to work? -->
-  <!-- TODO: Why isn't @jls a "standard" tag? -->
-  <!--   property name="javadoc.jls.cite" value="The Java&trade; Language Specification" -->
-  <property name="javadoc.jls.cite" value="The Java Language Specification"/>
+  <property name="javadoc.jls.cite" value="The Java&amp;trade; Language Specification"/>
   <property name="javadoc.jls.option" value="jls:a:See &lt;cite&gt;${javadoc.jls.cite}&lt;/cite&gt;:"/>
 
   <!-- Main targets -->
@@ -220,22 +222,14 @@
            fork="true">
 
       <include name="**/*.java"/>
-      <compilerarg line="${build.args}"/>
       <compilerarg value="-XDignore.symbol.file=true"/>
       <compilerarg value="-Xlint:all"/>
-
-<!--
-      <exclude name="java/lang/**"/>
       <compilerarg line="${build.args}"/>
-      <compilerarg line="-Xlint -Xmaxwarns 1000"/>
--->
 
     </javac>
-
   </target>
 
 
-
   <target name="jar"
           depends="compile"
           description="Builds library jar from compiled sources">
@@ -243,17 +237,14 @@
     <jar destfile="${product.jar}">
       <fileset dir="${build.classes.dir}"/>
     </jar>
-
   </target>
 
 
-
   <target name="test"
           depends="configure-tests, report-tests"
           description="Runs all tests (requires JUnit 3.8.1 in ${ant.home}/lib)" />
 
 
-
   <target name="docs"
           description="Builds javadocs for src/main to dist dir">
 
@@ -270,18 +261,16 @@
              sourcepath="${src.dir}:${jdk8src.dir}"
              classpath=""
              executable="${javadoc8}">
-      <arg value="-XDignore.symbol.file=true"/>
-      <arg value="-tag"/>
-      <arg value="${javadoc.jls.option}"/>
       <fileset dir="${src.dir}" defaultexcludes="yes">
         <include name="**/*.java"/>
       </fileset>
+      <arg value="-XDignore.symbol.file=true"/>
+      <arg value="-tag"/>
+      <arg value="${javadoc.jls.option}"/>
     </javadoc>
-
   </target>
 
 
-
   <target name="dist"
           depends="dist-clean, dist-jar, docs"
           description="Puts all distributable products in single hierarchy"/>
@@ -302,11 +291,9 @@
       <exclude name="**/SyntaxTest.java"/>
       <exclude name="**/SuperfluousAbstract.java"/>
     </jar>
-
   </target>
 
 
-
   <target name="clean"
           description="Removes all build products">
 
@@ -315,7 +302,6 @@
   </target>
 
 
-
   <target name="dist-clean"
           description="Removes all build and distribution products">
 
@@ -324,7 +310,6 @@
   </target>
 
 
-
   <!-- Internal targets -->
 
 
@@ -348,16 +333,16 @@
            deprecation="${build.deprecation}"
            source="6"
            classpath="${junit.jar}"
-           bootclasspath="${product.jar}:${bootclasspath6}"
+           bootclasspath="@{product.jar}:${bootclasspath6}"
            includeAntRuntime="false"
            includeJavaRuntime="false"
            executable="${javac8}"
            fork="true">
 
       <include name="**/*.java"/>
-      <compilerarg line="${build.args}"/>
       <compilerarg value="-XDignore.symbol.file=true"/>
       <compilerarg value="-Xlint:all,-unchecked,-rawtypes,-serial,-deprecation"/>
+      <compilerarg line="${build.args}"/>
 
     </javac>
 
@@ -368,17 +353,16 @@
            deprecation="${build.deprecation}"
            source="6"
            classpath=""
-           bootclasspath="${product.jar}:${bootclasspath6}"
+           bootclasspath="@{product.jar}:${bootclasspath6}"
            includeAntRuntime="false"
            includeJavaRuntime="false"
            executable="${javac8}"
            fork="true">
 
       <include name="jsr166/test/**/*.java"/>
-
-      <compilerarg line="${build.args}"/>
       <compilerarg value="-XDignore.symbol.file=true"/>
       <compilerarg value="-Xlint:all,-unchecked,-rawtypes,-serial,-deprecation"/>
+      <compilerarg line="${build.args}"/>
 
     </javac>
 
@@ -391,9 +375,9 @@
            source="${build.sourcelevel}"
              fork="true">
 
-      <compilerarg value="${bootclasspath.args}"/>
-      <compilerarg line="${build.args}"/>
+      <compilerarg value="-Xbootclasspath/p:${product.jar}"/>
       <compilerarg value="-XDignore.symbol.file=true"/>
+      <compilerarg line="${build.args}"/>
 
     </javac>
 -->
@@ -546,7 +530,6 @@
 
   <!-- jsr166 4jdk7 -->
 
-
   <target name="4jdk7compile"
           depends="configure-compiler"
           description="Compiles src/jdk7 sources, targeting jdk7">
@@ -567,14 +550,14 @@
            fork="true">
 
       <include name="**/*.java"/>
-      <compilerarg line="${build.args}"/>
       <compilerarg value="-XDignore.symbol.file=true"/>
       <compilerarg value="-Xlint:all"/>
+      <compilerarg line="${build.args}"/>
 
     </javac>
-
   </target>
 
+
   <target name="4jdk7-jar"
           depends="4jdk7compile"
           description="Builds library jar from compiled sources">
@@ -666,12 +649,11 @@
              sourcepath="${4jdk7src.dir}:${jdk7src.dir}"
              classpath=""
              executable="${javadoc7}">
-      <arg value="-XDignore.symbol.file=true"/>
       <fileset dir="${4jdk7src.dir}" defaultexcludes="yes">
         <include name="**/*.java"/>
       </fileset>
+      <arg value="-XDignore.symbol.file=true"/>
     </javadoc>
-
   </target>
 
 
@@ -680,7 +662,6 @@
           description="Puts all distributable products in single hierarchy"/>
 
 
-
   <target name="4jdk7clean"
           description="Removes all 4jdk7 build products">
 
@@ -689,14 +670,13 @@
   </target>
 
 
-
   <target name="4jdk7dist-clean"
           description="Removes all build and distribution products">
 
   </target>
 
   <target name="4jdk7dist-jar"
-          depends="4jdk7clean, 4jdk7compile">
+          depends="4jdk7clean, 4jdk7-jar">
 
     <copy file="${4jdk7product.jar}" todir="${dist.dir}"/>
 
@@ -717,25 +697,23 @@
            debuglevel="${build.debuglevel}"
            deprecation="${build.deprecation}"
            classpath=""
-           source="5"
            bootclasspath="${bootclasspath6}"
+           source="5"
            includeAntRuntime="false"
            includeJavaRuntime="false"
            executable="${javac7}"
            fork="true">
 
       <include name="jsr166x/**/*.java"/>
-      <compilerarg line="${build.args}"/>
       <compilerarg value="-XDignore.symbol.file=true"/>
       <compilerarg value="-Xlint:all,-unchecked,-rawtypes"/>
+      <compilerarg line="${build.args}"/>
 
     </javac>
-
   </target>
 
 
-
-  <target name="jsr166xjar"
+  <target name="jsr166x-jar"
           depends="jsr166xcompile"
           description="Builds library jar from compiled sources">
 
@@ -746,7 +724,6 @@
   </target>
 
 
-
   <target name="jsr166xdocs"
           description="Builds javadocs to dist dir">
 
@@ -756,12 +733,13 @@
     <javadoc destdir="${dist.jsr166xjavadocs.dir}"
              packagenames="jsr166x.*"
              link="${jdkapidocs.url}"
-             source="5"
-             bootclasspath="${bootclasspath6}"
              sourcepath="${topsrc.dir}:${jdk6src.dir}"
-             executable="${javadoc7}"
-             additionalparam="-XDignore.symbol.file=true"/>
+             bootclasspath="${bootclasspath6}"
+             source="5"
+             executable="${javadoc7}">
+      <arg value="-XDignore.symbol.file=true"/>
 
+    </javadoc>
   </target>
 
 
@@ -770,7 +748,6 @@
           description="Puts all distributable products in single hierarchy"/>
 
 
-
   <target name="jsr166xclean"
           description="Removes all jsr166x build products">
 
@@ -779,7 +756,6 @@
   </target>
 
 
-
   <target name="jsr166xdist-clean"
           description="Removes all build and distribution products">
 
@@ -787,7 +763,7 @@
 
 
   <target name="jsr166xdist-jar"
-          depends="jsr166xclean, jsr166xjar">
+          depends="jsr166xclean, jsr166x-jar">
 
     <copy file="${jsr166x.jar}" todir="${dist.dir}"/>
 
@@ -816,14 +792,15 @@
            fork="true">
 
       <include name="jsr166y/**/*.java"/>
-      <compilerarg line="${build.args}"/>
       <compilerarg value="-XDignore.symbol.file=true"/>
       <compilerarg value="-Xlint:all"/>
+      <compilerarg line="${build.args}"/>
+
     </javac>
   </target>
 
 
-  <target name="jsr166yjar"
+  <target name="jsr166y-jar"
           depends="jsr166ycompile"
           description="Builds library jar from compiled sources">
 
@@ -834,7 +811,6 @@
   </target>
 
 
-
   <target name="jsr166ydocs"
           description="Builds javadocs to dist dir">
 
@@ -844,12 +820,13 @@
     <javadoc destdir="${dist.jsr166yjavadocs.dir}"
              packagenames="jsr166y.*"
              link="${jdkapidocs.url}"
-             source="6"
-             bootclasspath="${bootclasspath6}"
              sourcepath="${topsrc.dir}:${jdk6src.dir}"
-             executable="${javadoc7}"
-             additionalparam="-XDignore.symbol.file=true"/>
+             bootclasspath="${bootclasspath6}"
+             source="6"
+             executable="${javadoc7}">
+      <arg value="-XDignore.symbol.file=true"/>
 
+    </javadoc>
   </target>
 
 
@@ -858,7 +835,6 @@
           description="Puts all distributable products in single hierarchy"/>
 
 
-
   <target name="jsr166yclean"
           description="Removes all jsr166y build products">
 
@@ -867,14 +843,13 @@
   </target>
 
 
-
   <target name="jsr166ydist-clean"
           description="Removes all build and distribution products">
 
   </target>
 
   <target name="jsr166ydist-jar"
-          depends="jsr166yclean, jsr166yjar">
+          depends="jsr166yclean, jsr166y-jar">
 
     <copy file="${jsr166y.jar}" todir="${dist.dir}"/>
 
@@ -885,7 +860,7 @@
 
 
   <target name="extra166ycompile"
-          depends="configure-compiler, jsr166yjar"
+          depends="configure-compiler, jsr166y-jar"
           description="Compiles extra166y sources">
 
     <mkdir dir="${build.extra166y.classes.dir}"/>
@@ -895,26 +870,24 @@
            debug="${build.debug}"
            debuglevel="${build.debuglevel}"
            deprecation="${build.deprecation}"
+           bootclasspath="@{jsr166y.jar}:${bootclasspath6}"
            classpath=""
            source="6"
-           bootclasspath="${jsr166y.jar}:${bootclasspath6}"
            includeAntRuntime="false"
            includeJavaRuntime="false"
            executable="${javac7}"
            fork="true">
 
       <include name="extra166y/**/*.java"/>
-      <compilerarg line="${build.args}"/>
       <compilerarg value="-XDignore.symbol.file=true"/>
       <compilerarg value="-Xlint:all,-unchecked,-rawtypes,-serial"/>
+      <compilerarg line="${build.args}"/>
 
     </javac>
-
   </target>
 
 
-
-  <target name="extra166yjar"
+  <target name="extra166y-jar"
           depends="extra166ycompile"
           description="Builds library jar from compiled sources">
 
@@ -925,7 +898,6 @@
   </target>
 
 
-
   <target name="extra166ydocs"
           description="Builds javadocs to build dir">
 
@@ -935,12 +907,13 @@
     <javadoc destdir="${dist.extra166yjavadocs.dir}"
              packagenames="extra166y.*"
              link="${jdkapidocs.url}"
-             source="6"
-             bootclasspath="${bootclasspath6}"
              sourcepath="${topsrc.dir}:${jdk6src.dir}"
-             executable="${javadoc7}"
-             additionalparam="-XDignore.symbol.file=true"/>
+             bootclasspath="${bootclasspath6}"
+             source="6"
+             executable="${javadoc7}">
+      <arg value="-XDignore.symbol.file=true"/>
 
+    </javadoc>
   </target>
 
 
@@ -949,7 +922,6 @@
           description="Puts all distributable products in single hierarchy"/>
 
 
-
   <target name="extra166yclean"
           description="Removes all extra166y build products">
 
@@ -958,14 +930,13 @@
   </target>
 
 
-
   <target name="extra166ydist-clean"
           description="Removes all build and distribution products">
 
   </target>
 
   <target name="extra166ydist-jar"
-          depends="extra166yclean, extra166yjar">
+          depends="extra166yclean, extra166y-jar">
 
     <copy file="${extra166y.jar}" todir="${dist.dir}"/>
 
@@ -992,17 +963,15 @@
            fork="true">
 
       <include name="jsr166e/**/*.java"/>
-      <compilerarg line="${build.args}"/>
       <compilerarg value="-XDignore.symbol.file=true"/>
       <compilerarg value="-Xlint:all"/>
+      <compilerarg line="${build.args}"/>
 
     </javac>
-
   </target>
 
 
-
-  <target name="jsr166ejar"
+  <target name="jsr166e-jar"
           depends="jsr166ecompile"
           description="Builds library jar from compiled sources">
 
@@ -1013,7 +982,6 @@
   </target>
 
 
-
   <target name="jsr166edocs"
           description="Builds javadocs to build dir">
 
@@ -1023,12 +991,12 @@
     <javadoc destdir="${dist.jsr166ejavadocs.dir}"
              packagenames="jsr166e.*"
              link="${jdkapidocs.url}"
-             source="7"
              sourcepath="${topsrc.dir}:${jdk7src.dir}"
-             executable="${javadoc7}"
-             additionalparam="-XDignore.symbol.file=true" >
-    </javadoc>
+             source="7"
+             executable="${javadoc7}">
+      <arg value="-XDignore.symbol.file=true"/>
 
+    </javadoc>
   </target>
 
 
@@ -1037,7 +1005,6 @@
           description="Puts all distributable products in single hierarchy"/>
 
 
-
   <target name="jsr166eclean"
           description="Removes all jsr166e build products">
 
@@ -1052,7 +1019,7 @@
   </target>
 
   <target name="jsr166edist-jar"
-          depends="jsr166eclean, jsr166ejar">
+          depends="jsr166eclean, jsr166e-jar">
 
     <copy file="${jsr166e.jar}" todir="${dist.dir}"/>