New filesonly attribute for <zip> and friends - suppresses directories.

PR: 2053
Submitted by:	Peter Janes <peterj@liberate.com>


git-svn-id: https://svn.apache.org/repos/asf/ant/core/trunk@269324 13f79535-47bb-0310-9956-ffa450edef68

WHATSNEW

@@ -102,6 +102,9 @@ Other changes:
 
 * regexp mapper now supports the java.util.regex package of JDK 1.4.
 
+* New filesonly attribute for <zip> and friends to suppress directory
+  entries.
+
 Fixed bugs:
 -----------
 

docs/manual/CoreTasks/ear.html

@@ -51,6 +51,11 @@ attributes of zipfilesets in a Zip or Jar task.)</p>
       likely be unreadable for Java otherwise.</strong></td>
     <td align="center" valign="top">No</td>
   </tr>
+  <tr>
+    <td valign="top">filesonly</td>
+    <td valign="top">Store only file entries, defaults to false</td>
+    <td align="center" valign="top">No</td>
+  </tr>
   <tr>
     <td valign="top">includes</td>
     <td valign="top">comma separated list of patterns of files that must be

docs/manual/CoreTasks/jar.html

@@ -73,6 +73,11 @@ include an empty one for you.)</p>
       likely be unreadable for Java otherwise.</strong></td>
     <td align="center" valign="top">No</td>
   </tr>
+  <tr>
+    <td valign="top">filesonly</td>
+    <td valign="top">Store only file entries, defaults to false</td>
+    <td align="center" valign="top">No</td>
+  </tr>
   <tr>
     <td valign="top">includes</td>
     <td valign="top">comma separated list of patterns of files that must be

docs/manual/CoreTasks/war.html

@@ -53,6 +53,11 @@ attributes of zipfilesets in a Zip or Jar task.)</p>
       likely be unreadable for Java otherwise.</strong></td>
     <td align="center" valign="top">No</td>
   </tr>
+  <tr>
+    <td valign="top">filesonly</td>
+    <td valign="top">Store only file entries, defaults to false</td>
+    <td align="center" valign="top">No</td>
+  </tr>
   <tr>
     <td valign="top">includes</td>
     <td valign="top">comma separated list of patterns of files that must be

docs/manual/CoreTasks/zip.html

@@ -74,6 +74,11 @@ Java.</p>
     Defaults to the platform's default character encoding.</td>
     <td align="center" valign="top">No</td>
   </tr>
+  <tr>
+    <td valign="top">filesonly</td>
+    <td valign="top">Store only file entries, defaults to false</td>
+    <td align="center" valign="top">No</td>
+  </tr>
   <tr>
     <td valign="top">includes</td>
     <td valign="top">comma separated list of patterns of files that must be

src/main/org/apache/tools/ant/taskdefs/Zip.java

@@ -78,6 +78,7 @@ public class Zip extends MatchingTask {
     private File zipFile;
     private File baseDir;
     private boolean doCompress = true;
+    private boolean doFilesonly = false;
     protected String archiveType = "zip";
     // For directories:
     private static long emptyCrc = new CRC32 ().getValue ();
@@ -114,6 +115,13 @@ public class Zip extends MatchingTask {
         doCompress = c;
     }
 
+    /**
+     * Emulate Sun's jar utility by not adding parent dirs
+     */
+    public void setFilesonly(boolean f) {
+        doFilesonly = f;
+    }
+
     /**
      * Adds a set of files (nested fileset attribute).
      */
@@ -529,27 +537,28 @@ public class Zip extends MatchingTask {
     protected void addParentDirs(File baseDir, String entry,
                                  ZipOutputStream zOut, String prefix)
         throws IOException {
-
-        Stack directories = new Stack();
-        int slashPos = entry.length();
-
-        while ((slashPos = entry.lastIndexOf((int)'/', slashPos-1)) != -1) {
-            String dir = entry.substring(0, slashPos+1);
-            if (addedDirs.get(prefix+dir) != null) {
-                break;
+        if( !doFilesonly ) {
+            Stack directories = new Stack();
+            int slashPos = entry.length();
+            
+            while ((slashPos = entry.lastIndexOf((int)'/', slashPos-1)) != -1) {
+                String dir = entry.substring(0, slashPos+1);
+                if (addedDirs.get(prefix+dir) != null) {
+                    break;
+                }
+                directories.push(dir);
             }
-            directories.push(dir);
-        }
-
-        while (!directories.isEmpty()) {
-            String dir = (String) directories.pop();
-            File f = null;
-            if (baseDir != null) {
-                f = new File(baseDir, dir);
-            } else {
-                f = new File(dir);
+            
+            while (!directories.isEmpty()) {
+                String dir = (String) directories.pop();
+                File f = null;
+                if (baseDir != null) {
+                    f = new File(baseDir, dir);
+                } else {
+                    f = new File(dir);
+                }
+                zipDir(f, zOut, prefix+dir);
             }
-            zipDir(f, zOut, prefix+dir);
         }
     }