 Chromium Code Reviews
 Chromium Code Reviews Issue 10905138:
  Add test jar generation logic for ant builds.  (Closed) 
  Base URL: http://git.chromium.org/chromium/src.git@master
    
  
    Issue 10905138:
  Add test jar generation logic for ant builds.  (Closed) 
  Base URL: http://git.chromium.org/chromium/src.git@master| OLD | NEW | 
|---|---|
| 1 <!-- | 1 <!-- | 
| 2 Copyright (c) 2012 The Chromium Authors. All rights reserved. | 2 Copyright (c) 2012 The Chromium Authors. All rights reserved. | 
| 3 Use of this source code is governed by a BSD-style license that can be | 3 Use of this source code is governed by a BSD-style license that can be | 
| 4 found in the LICENSE file. | 4 found in the LICENSE file. | 
| 5 --> | 5 --> | 
| 6 <project name="chromium-jars" default="dist"> | 6 <project name="chromium-jars" default="dist"> | 
| 7 <!-- | 7 <!-- | 
| 8 Common ant build file for for chromium_*.jars. | 8 Common ant build file for for chromium_*.jars. | 
| 9 For creating a new chromium_*.jar : | 9 For creating a new chromium_*.jar : | 
| 10 1. Use build/java.gypi action. | 10 1. Use build/java.gypi action. | 
| (...skipping 17 matching lines...) Expand all Loading... | |
| 28 | 28 | 
| 29 <property-value | 29 <property-value | 
| 30 name="javac.srcdir" | 30 name="javac.srcdir" | 
| 31 value="src:${toString:javac.srcdirs.additional}" | 31 value="src:${toString:javac.srcdirs.additional}" | 
| 32 /> | 32 /> | 
| 33 | 33 | 
| 34 <property-location | 34 <property-location | 
| 35 name="lib.dir" | 35 name="lib.dir" | 
| 36 location="${PRODUCT_DIR}/lib.java" | 36 location="${PRODUCT_DIR}/lib.java" | 
| 37 check-exists="false" | 37 check-exists="false" | 
| 38 /> | 38 /> | 
| 
cjhopman
2012/09/06 22:33:41
This property is no longer used.
 
shashi
2012/09/07 00:41:33
Done.
 | |
| 39 | 39 | 
| 40 <property-location | 40 <property-location | 
| 41 name="dest.dir" | 41 name="dest.dir" | 
| 42 location="${PRODUCT_DIR}/java/${PACKAGE_NAME}" | 42 location="${PRODUCT_DIR}/java/${PACKAGE_NAME}" | 
| 43 check-exists="false" | 43 check-exists="false" | 
| 44 /> | 44 /> | 
| 45 | 45 | 
| 46 <target name="init"> | 46 <target name="init"> | 
| 47 <!-- Create the time stamp --> | 47 <!-- Create the time stamp --> | 
| 48 <tstamp/> | 48 <tstamp/> | 
| 49 <!-- Create the build directory structure used by compile --> | 49 <!-- Create the build directory structure used by compile --> | 
| 50 <mkdir dir="${lib.dir}"/> | 50 <mkdir dir="${lib.java.dir}"/> | 
| 
cjhopman
2012/09/06 22:33:41
Remove the mkdir here. Should now be made by the n
 
shashi
2012/09/07 00:41:33
Done.
 | |
| 51 <mkdir dir="${dest.dir}"/> | 51 <mkdir dir="${dest.dir}"/> | 
| 52 | 52 | 
| 53 <!-- Remove all .class files from dest.dir. This prevents inclusion of | 53 <!-- Remove all .class files from dest.dir. This prevents inclusion of | 
| 54 incorrect .class files in the final .jar. For example, if a .java file | 54 incorrect .class files in the final .jar. For example, if a .java file | 
| 55 was deleted, the .jar should not contain the .class files for that | 55 was deleted, the .jar should not contain the .class files for that | 
| 56 .java from previous builds. | 56 .java from previous builds. | 
| 57 --> | 57 --> | 
| 58 <delete> | 58 <delete> | 
| 59 <fileset dir="${dest.dir}" includes="**/*.class"/> | 59 <fileset dir="${dest.dir}" includes="**/*.class"/> | 
| 60 </delete> | 60 </delete> | 
| (...skipping 17 matching lines...) Expand all Loading... | |
| 78 classpathref="javac.custom.classpath" | 78 classpathref="javac.custom.classpath" | 
| 79 debug="true" | 79 debug="true" | 
| 80 includeantruntime="false" | 80 includeantruntime="false" | 
| 81 /> | 81 /> | 
| 82 </target> | 82 </target> | 
| 83 | 83 | 
| 84 <target name="dist" depends="compile" | 84 <target name="dist" depends="compile" | 
| 85 description="Generate chromium_${PACKAGE_NAME}.jar."> | 85 description="Generate chromium_${PACKAGE_NAME}.jar."> | 
| 86 <!-- Create the distribution directory --> | 86 <!-- Create the distribution directory --> | 
| 87 <jar | 87 <jar | 
| 88 jarfile="${lib.dir}/chromium_${PACKAGE_NAME}.jar" | 88 jarfile="${lib.dir}/chromium_${PACKAGE_NAME}.jar" | 
| 
cjhopman
2012/09/06 22:33:41
This should be lib.java.dir now.
 
shashi
2012/09/07 00:41:33
Done.
 | |
| 89 basedir="${dest.dir}" | 89 basedir="${dest.dir}" | 
| 90 /> | 90 /> | 
| 91 | 91 | 
| 92 <!-- If Gyp thinks this output is stale but Ant doesn't, the modification | 92 <!-- If Gyp thinks this output is stale but Ant doesn't, the modification | 
| 93 time should still be updated. Otherwise, this target will continue to | 93 time should still be updated. Otherwise, this target will continue to | 
| 94 be rebuilt in future builds. | 94 be rebuilt in future builds. | 
| 95 --> | 95 --> | 
| 96 <touch file="${lib.dir}/chromium_${PACKAGE_NAME}.jar"/> | 96 <touch file="${lib.java.dir}/chromium_${PACKAGE_NAME}.jar"/> | 
| 97 </target> | 97 </target> | 
| 98 | 98 | 
| 99 <target name="clean" description="clean up"> | 99 <target name="clean" description="clean up"> | 
| 100 <!-- Delete the appropriate directory trees --> | 100 <!-- Delete the appropriate directory trees --> | 
| 101 <delete dir="${dest.dir}" /> | 101 <delete dir="${dest.dir}" /> | 
| 102 </target> | 102 </target> | 
| 103 </project> | 103 </project> | 
| OLD | NEW |