MANIFEST.MF, the file is
ignored and you will get a warning.
- <ear file="${build.dir}/myapp.ear" appxml="${src.dir}/metadata/application.xml">
+ <ear destfile="${build.dir}/myapp.ear" appxml="${src.dir}/metadata/application.xml">
<fileset dir="${build.dir}" includes="*.jar,*.war"/>
</ear>
diff --git a/docs/manual/CoreTasks/jar.html b/docs/manual/CoreTasks/jar.html
index 0eab3320a..4182d7258 100644
--- a/docs/manual/CoreTasks/jar.html
+++ b/docs/manual/CoreTasks/jar.html
@@ -58,8 +58,8 @@ being wrapped and continued on the next line.
<jar file="${dist}/lib/app.jar" basedir="${build}/classes"/>
jars all files in the ${build}/classes directory into a file
called app.jar in the ${dist}/lib directory.
<jar file="${dist}/lib/app.jar"
+ <jar destfile="${dist}/lib/app.jar"
basedir="${build}/classes"
excludes="**/Test.class"
/>
jars all files in the ${build}/classes directory into a file
called app.jar in the ${dist}/lib directory. Files
with the name Test.class are excluded.
- <jar file="${dist}/lib/app.jar"
+ <jar destfile="${dist}/lib/app.jar"
basedir="${build}/classes"
includes="mypackage/test/**"
excludes="**/Test.class"
@@ -173,7 +173,7 @@ with the name Test.class are excluded.
called app.jar in the ${dist}/lib directory. Only
files under the directory mypackage/test are used, and files with
the name Test.class are excluded.
- <jar file="${dist}/lib/app.jar">
+ <jar destfile="${dist}/lib/app.jar">
<fileset dir="${build}/classes"
excludes="**/Test.class"
/>
@@ -188,7 +188,7 @@ and ${src}/resources/mypackage/image.gif, they will appear
in the same directory in the JAR (and thus be considered in the same package
by Java).
- <jar jarfile="test.jar" basedir=".">
+ <jar destfile="test.jar" basedir=".">
<include name="build"/>
<manifest>
<attribute name="Built-By" value="${user.name}"/>
diff --git a/docs/manual/CoreTasks/war.html b/docs/manual/CoreTasks/war.html
index 1e29b8b64..1c66520cc 100644
--- a/docs/manual/CoreTasks/war.html
+++ b/docs/manual/CoreTasks/war.html
@@ -26,8 +26,8 @@ attributes of zipfilesets in a Zip or Jar task.)
Required
myapp.war created with
-<war file="myapp.war" webxml="src/metadata/myapp.xml">
+<war destfile="myapp.war" webxml="src/metadata/myapp.xml">
<fileset dir="src/html/myapp"/>
<fileset dir="src/jsp/myapp"/>
<lib dir="thirdparty/libs">
diff --git a/docs/manual/CoreTasks/zip.html b/docs/manual/CoreTasks/zip.html
index f4e762502..3eb5349ed 100644
--- a/docs/manual/CoreTasks/zip.html
+++ b/docs/manual/CoreTasks/zip.html
@@ -57,7 +57,7 @@ Java.
Required
<zip file="${dist}/manual.zip"
+ <zip destfile="${dist}/manual.zip"
basedir="htdocs/manual"
/>
zips all files in the htdocs/manual directory into a file called manual.zip
in the ${dist} directory.
- <zip file="${dist}/manual.zip"
+ <zip destfile="${dist}/manual.zip"
basedir="htdocs/manual"
update="true"
/>
@@ -155,27 +155,27 @@ in the ${dist} directory.
in the ${dist} directory. If manual.zip
doesn't exist, it is created; otherwise it is updated with the
new/changed files.
- <zip file="${dist}/manual.zip"
+ <zip destfile="${dist}/manual.zip"
basedir="htdocs/manual"
excludes="mydocs/**, **/todo.html"
/>
zips all files in the htdocs/manual directory. Files in the directory mydocs,
or files with the name todo.html are excluded.
- <zip file="${dist}/manual.zip"
+ <zip destfile="${dist}/manual.zip"
basedir="htdocs/manual"
includes="api/**/*.html"
excludes="**/todo.html"
/>
zips all files in the htdocs/manual directory. Only html files under the directory api
are zipped, and files with the name todo.html are excluded.
- <zip file="${dist}/manual.zip">
+ <zip destfile="${dist}/manual.zip">
<fileset dir="htdocs/manual"/>
<fileset dir="." includes="ChangeLog.txt"/>
</zip>
zips all files in the htdocs/manual directory, and also adds the file ChangeLog.txt in the
current directory. ChangeLog.txt will be added to the top of the ZIP file, just as if
it had been located at htdocs/manual/ChangeLog.txt.
- <zip file="${dist}/manual.zip">
+ <zip destfile="${dist}/manual.zip">
<zipfileset dir="htdocs/manual" prefix="docs/user-guide"/>
<zipfileset dir="." includes="ChangeLog27.txt" fullpath="docs/ChangeLog.txt"/>
<zipfileset src="examples.zip" includes="**/*.html" prefix="docs/examples"/>
diff --git a/src/etc/testcases/taskdefs/jar.xml b/src/etc/testcases/taskdefs/jar.xml
index 064c1fe97..81679a558 100644
--- a/src/etc/testcases/taskdefs/jar.xml
+++ b/src/etc/testcases/taskdefs/jar.xml
@@ -15,7 +15,7 @@
@@ -24,7 +24,7 @@
-
\ No newline at end of file
+
diff --git a/src/etc/testcases/taskdefs/zip.xml b/src/etc/testcases/taskdefs/zip.xml
index 66bd77e71..ec48043d5 100644
--- a/src/etc/testcases/taskdefs/zip.xml
+++ b/src/etc/testcases/taskdefs/zip.xml
@@ -7,21 +7,21 @@
-
-
@@ -32,28 +32,35 @@
-
+
-
+
-
+
-
+
+
+
+
+
+
element.
- if (vPath.equalsIgnoreCase("META-INF/aplication.xml")) {
+ if (vPath.equalsIgnoreCase("META-INF/application.xml")) {
if (deploymentDescriptor == null || !deploymentDescriptor.equals(file) || descriptorAdded) {
log("Warning: selected "+archiveType+" files include a META-INF/application.xml which will be ignored " +
"(please use appxml attribute to "+archiveType+" task)", Project.MSG_WARN);
@@ -139,7 +145,7 @@ public class Ear extends Jar {
}
/**
- * Make sure we don't think we already have a web.xml next time this task
+ * Make sure we don't think we already have a application.xml next time this task
* gets executed.
*/
protected void cleanUp() {
diff --git a/src/main/org/apache/tools/ant/taskdefs/Jar.java b/src/main/org/apache/tools/ant/taskdefs/Jar.java
index ae40a674a..67eb15b82 100644
--- a/src/main/org/apache/tools/ant/taskdefs/Jar.java
+++ b/src/main/org/apache/tools/ant/taskdefs/Jar.java
@@ -97,11 +97,11 @@ public class Jar extends Zip {
}
/**
- * @deprecated use setFile(File) instead.
+ * @deprecated Use setDestFile(File) instead
*/
public void setJarfile(File jarFile) {
- log("DEPRECATED - The jarfile attribute is deprecated. Use file attribute instead.");
- setFile(jarFile);
+ log("DEPRECATED - The jarfile attribute is deprecated. Use destfile attribute instead.");
+ setDestFile(jarFile);
}
/**
diff --git a/src/main/org/apache/tools/ant/taskdefs/War.java b/src/main/org/apache/tools/ant/taskdefs/War.java
index 918b36f40..333116a0c 100644
--- a/src/main/org/apache/tools/ant/taskdefs/War.java
+++ b/src/main/org/apache/tools/ant/taskdefs/War.java
@@ -79,11 +79,17 @@ public class War extends Jar {
emptyBehavior = "create";
}
+ /**
+ * @deprecated Use setDestFile(File) instead
+ */
public void setWarfile(File warFile) {
- log("DEPRECATED - The warfile attribute is deprecated. Use file attribute instead.");
- setFile(warFile);
+ log("DEPRECATED - The warfile attribute is deprecated. Use destfile attribute instead.");
+ setDestFile(warFile);
}
+ /**
+ * set the web app descriptor for this WAR file
+ */
public void setWebxml(File descr) {
deploymentDescriptor = descr;
if (!deploymentDescriptor.exists())
diff --git a/src/main/org/apache/tools/ant/taskdefs/Zip.java b/src/main/org/apache/tools/ant/taskdefs/Zip.java
index 050663a33..1c2adc183 100644
--- a/src/main/org/apache/tools/ant/taskdefs/Zip.java
+++ b/src/main/org/apache/tools/ant/taskdefs/Zip.java
@@ -120,22 +120,38 @@ public class Zip extends MatchingTask {
* This is the name/location of where to
* create the .zip file.
*
- * @deprecated Use setFile() instead
+ * @deprecated Use setDestFile(File) instead
*/
public void setZipfile(File zipFile) {
- log("DEPRECATED - The zipfile attribute is deprecated. Use file attribute instead.");
- setFile( zipFile );
+ log("DEPRECATED - The zipfile attribute is deprecated. Use destfile attribute instead.");
+ setDestFile(zipFile);
}
/**
* This is the name/location of where to
* create the .zip file.
- *
+ * @since 1.5alpha
+ * @todo pull this before shipping 1.5
+ * @deprecated Use setDestFile(File) instead
*/
public void setFile(File file) {
- this.zipFile = file;
+ log("DEPRECATED - The file attribute has been renamed destfile."
+ +" This attribute will be unsupported before Ant1.5 is released",
+ Project.MSG_ERR);
+ setDestFile(file);
+ }
+
+
+ /**
+ * Sets the destfile attribute.
+ * @since 1.5
+ * @param destFile The new destination File
+ */
+ public void setDestFile(File destFile) {
+ this.zipFile = destFile;
}
+
/**
* This is the base directory to look in for
* things to zip.
diff --git a/src/testcases/org/apache/tools/ant/taskdefs/ZipTest.java b/src/testcases/org/apache/tools/ant/taskdefs/ZipTest.java
index 9cd471c48..e85a91de4 100644
--- a/src/testcases/org/apache/tools/ant/taskdefs/ZipTest.java
+++ b/src/testcases/org/apache/tools/ant/taskdefs/ZipTest.java
@@ -101,4 +101,8 @@ public class ZipTest extends BuildFileTest {
public void test7() {
executeTarget("test7");
}
+
+ public void test8() {
+ executeTarget("test8");
+ }
}