Merge branch '1.9.x'
authorStefan Bodewig <bodewig@apache.org>
Sat, 12 Jan 2019 16:18:30 +0000 (17:18 +0100)
committerStefan Bodewig <bodewig@apache.org>
Sat, 12 Jan 2019 16:18:30 +0000 (17:18 +0100)
1  2 
WHATSNEW
src/main/org/apache/tools/ant/taskdefs/optional/clearcase/CCCheckout.java
src/main/org/apache/tools/ant/taskdefs/optional/clearcase/ClearCase.java

diff --cc WHATSNEW
+++ b/WHATSNEW
@@@ -4,17 -4,10 +4,21 @@@ Changes from Ant 1.10.5 TO Ant 1.10.
  Changes that could break older environments:
  -------------------------------------------
  
 + * image task no longer works on Java 9+ because internal classes
 +   supporting Java Advanced Imaging are removed; imageio task (based on
 +   ImageIO and AWT) is provided as a replacement.
 +
 + * junitlauncher task has changed the class names and package names of
 +   the task as well as some of the supporting classes of that task. If
 +   any code depended on these class or package names, they will have to
 +   be updated to reference these newly named classes. This however,
 +   doesn't impact build scripts if their reference to junitlauncher task
 +   was merely through the use of the <junitlauncher> element.
 +
+  * ClearCase#runS has been augmented by a two arg-version and the
+    one-arg version will no longer be called. This may affect
+    subclasses that have overridden runS.
  Fixed bugs:
  -----------
  
     an incorrect compression level for a zip entry. This is now fixed.
     Bugzilla Report 62686
  
 + * sync task, in some cases on case insensitive file systems, would consider
 +   a file in a destination directory to be orphaned and would delete it.
 +   This task has now been fixed to infer the case sensitivity of the filesystem
 +   of the destination directory.
 +   Bugzilla Report 62890
 +
 + * Fixes a potential java.util.ConcurrentModificationException in
 +   org.apache.tools.ant.Project#getCopyOfReferences.
 +   Github Pull Request #81
 +
+  * cccheckout would ignore an error of the "ls checkout" command even
+    if failOnError was set to false.
+    Bugzilla Report 63071
  
  Other changes:
  --------------
@@@ -217,11 -168,12 +217,11 @@@ public class CCCheckout extends ClearCa
          // viewpath
          cmdl.createArgument().setValue(getViewPath());
  
-         String result = runS(cmdl);
 -        result = runS(cmdl, getFailOnErr());
 -
 -        // System.out.println( "lsCheckout: " + result );
++        String result = runS(cmdl, getFailOnErr());
  
 -        return (result != null && result.length() > 0) ? true : false;
 +        return result != null && !result.isEmpty();
      }
 +
      /**
       * Check the command line options.
       */
@@@ -205,8 -144,21 +205,21 @@@ public abstract class ClearCase extend
       * Execute the given command, and return it's output
       * @param cmdline command line to execute
       * @return output of the command line
+      * @deprecated use the two arg version instead
       */
+     @Deprecated
      protected String runS(Commandline cmdline) {
 -     * @since Ant 1.9.14
+         return runS(cmdline, false);
+     }
+     /**
+      * Execute the given command, and return it's output
+      * @param cmdline command line to execute
+      * @param failOnError whether to fail the build if the command fails
+      * @return output of the command line
++     * @since Ant 1.10.6
+      */
+     protected String runS(Commandline cmdline, boolean failOnError) {
          String   outV  = "opts.cc.runS.output" + pcnt++;
          ExecTask exe   = new ExecTask(this);
          Commandline.Argument arg = exe.createArg();