You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by hi...@apache.org on 2010/03/19 21:10:15 UTC

svn commit: r925403 - /harmony/enhanced/jdktools/trunk/modules/jdktools/build.xml

Author: hindessm
Date: Fri Mar 19 20:10:15 2010
New Revision: 925403

URL: http://svn.apache.org/viewvc?rev=925403&view=rev
Log:
Commit r925287 "Apply partial jdktools patch for HARMONY-6477
(Problems building jdktools modules in Eclipse)." uses a variable
bcel.jar.url which is not defined.  This caused a build break.  This
commit fixes this.

It also added a depends check so I've added a corresponding fetch.

Modified:
    harmony/enhanced/jdktools/trunk/modules/jdktools/build.xml

Modified: harmony/enhanced/jdktools/trunk/modules/jdktools/build.xml
URL: http://svn.apache.org/viewvc/harmony/enhanced/jdktools/trunk/modules/jdktools/build.xml?rev=925403&r1=925402&r2=925403&view=diff
==============================================================================
--- harmony/enhanced/jdktools/trunk/modules/jdktools/build.xml (original)
+++ harmony/enhanced/jdktools/trunk/modules/jdktools/build.xml Fri Mar 19 20:10:15 2010
@@ -39,7 +39,7 @@
                         dest="${jdt.jdimodel.jar}" />
         <check-one-file src="${icu.url}" dest="${icu.jar}" />
         <check-one-file src="${ecj.jar.url}" dest="${ecj.jar}" />
-        <check-one-file src="${bcel.jar.url}" dest="${bcel.jar}" />
+        <check-one-file src="${bcel.jar}" dest="${bcel.jar}" />
         <copy todir="${hy.hdk}/jdk/jre/lib/boot">
             <fileset dir="${depends.jars}">
                 <patternset includes="${bcel.version}/*.jar" />
@@ -51,6 +51,9 @@
     <target name="fetch-depends">
         <fetch dep="jdt.jdi.jar"/>
         <fetch dep="jdt.jdimodel.jar"/>
+        <mkdir dir="${bcel.dir}" />
+        <download-one-file dest="${bcel.jar}" 
+                           src="${bcel.url}" md5="${bcel.md5}"/>
     </target>
 
     <target name="build" depends="check-depends, build-java" />