You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@uima.apache.org by ch...@apache.org on 2013/04/03 18:31:33 UTC

svn commit: r1464091 - /uima/sandbox/uima-ducc/trunk/uima-ducc-cli/src/main/java/org/apache/uima/ducc/cli/CliBase.java

Author: challngr
Date: Wed Apr  3 16:31:32 2013
New Revision: 1464091

URL: http://svn.apache.org/r1464091
Log:
UIMA-2792
Don't trust mkdirs() rc to test for directory creation, instead ignore the rc
and test the directory using File.exists();

Modified:
    uima/sandbox/uima-ducc/trunk/uima-ducc-cli/src/main/java/org/apache/uima/ducc/cli/CliBase.java

Modified: uima/sandbox/uima-ducc/trunk/uima-ducc-cli/src/main/java/org/apache/uima/ducc/cli/CliBase.java
URL: http://svn.apache.org/viewvc/uima/sandbox/uima-ducc/trunk/uima-ducc-cli/src/main/java/org/apache/uima/ducc/cli/CliBase.java?rev=1464091&r1=1464090&r2=1464091&view=diff
==============================================================================
--- uima/sandbox/uima-ducc/trunk/uima-ducc-cli/src/main/java/org/apache/uima/ducc/cli/CliBase.java (original)
+++ uima/sandbox/uima-ducc/trunk/uima-ducc-cli/src/main/java/org/apache/uima/ducc/cli/CliBase.java Wed Apr  3 16:31:32 2013
@@ -381,7 +381,9 @@ public abstract class CliBase
         String directory = props.getProperty("log_directory") + File.separator + friendlyId;
         String fileName = directory + File.separator + name;
         File f = new File(directory);
-        if ( ! f.mkdirs() ) {
+
+        f.mkdirs();
+        if ( ! f.exists() ) {
             throw new IllegalStateException("Cannot create log directory: " + f.toString());
         }