#45902: <zip> (without dofilesonly="true") sometimes recreated archives unnecessarily.

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

WHATSNEW

@@ -132,6 +132,8 @@ Changes that could break older environments:
 Fixed bugs:
 -----------
 
+ * <zip> would sometimes recreate JARs unnecessarily. Bugzilla report 45902.
+
  * <symlink> task couldn't overwrite existing symlinks that pointed to nonexistent files
    Bugzilla report 38199.
 

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

@@ -1238,9 +1238,7 @@ public class Zip extends MatchingTask {
             }
 
             Resource[] resources = initialResources[i];
-            if (doFilesonly) {
-                resources = selectFileResources(resources);
-            }
+            resources = selectFileResources(resources);
 
             newerResources[i] =
                 ResourceUtils.selectOutOfDateSources(this,
@@ -1331,9 +1329,7 @@ public class Zip extends MatchingTask {
             }
 
             Resource[] rs = initialResources[i];
-            if (doFilesonly) {
-                rs = selectFileResources(rs);
-            }
+            rs = selectFileResources(rs);
 
             newerResources[i] =
                 ResourceUtils.selectOutOfDateSources(this,