Author: sebb
Date: Thu Jun  4 14:05:19 2009
New Revision: 781741

URL: http://svn.apache.org/viewvc?rev=781741&view=rev
Log:
Change ResultCollector to only warn if the directory was not created

Modified:
    
jakarta/jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java
    jakarta/jmeter/trunk/xdocs/changes.xml

Modified: 
jakarta/jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java
URL: 
http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java?rev=781741&r1=781740&r2=781741&view=diff
==============================================================================
--- 
jakarta/jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java 
(original)
+++ 
jakarta/jmeter/trunk/src/core/org/apache/jmeter/reporters/ResultCollector.java 
Thu Jun  4 14:05:19 2009
@@ -375,7 +375,8 @@
             // and create it - if there is one
             File pdir = new File(filename).getParentFile();
             if (pdir != null) {
-                if (!pdir.mkdirs()){
+                pdir.mkdirs();// returns false if directory already exists, so 
need to check again
+                if (!pdir.exists()){
                     log.warn("Error creating directories for 
"+pdir.toString());
                 }
             }

Modified: jakarta/jmeter/trunk/xdocs/changes.xml
URL: 
http://svn.apache.org/viewvc/jakarta/jmeter/trunk/xdocs/changes.xml?rev=781741&r1=781740&r2=781741&view=diff
==============================================================================
--- jakarta/jmeter/trunk/xdocs/changes.xml (original)
+++ jakarta/jmeter/trunk/xdocs/changes.xml Thu Jun  4 14:05:19 2009
@@ -78,6 +78,7 @@
 
 <h3>Listeners</h3>
 <ul>
+<li>Change ResultCollector to only warn if the directory was not created</li>
 </ul>
 
 <h3>Assertions</h3>



---------------------------------------------------------------------
To unsubscribe, e-mail: jmeter-dev-unsubscr...@jakarta.apache.org
For additional commands, e-mail: jmeter-dev-h...@jakarta.apache.org

Reply via email to