Author: krosenvold
Date: Mon Aug 6 18:46:59 2012
New Revision: 1369919
URL: http://svn.apache.org/viewvc?rev=1369919&view=rev
Log:
[SUREFIRE-846] Do not leave target/surefire folder around unless debug logging
is on
Patch by Martin Todorov, applied with some changes
Modified:
maven/surefire/trunk/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java
Modified:
maven/surefire/trunk/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java
URL:
http://svn.apache.org/viewvc/maven/surefire/trunk/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java?rev=1369919&r1=1369918&r2=1369919&view=diff
==============================================================================
---
maven/surefire/trunk/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java
(original)
+++
maven/surefire/trunk/maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java
Mon Aug 6 18:46:59 2012
@@ -81,6 +81,7 @@ import org.apache.maven.surefire.util.Ru
import org.apache.maven.surefire.util.ScanResult;
import org.apache.maven.toolchain.Toolchain;
import org.apache.maven.toolchain.ToolchainManager;
+import org.codehaus.plexus.util.FileUtils;
import org.codehaus.plexus.util.StringUtils;
/**
@@ -703,6 +704,26 @@ public abstract class AbstractSurefireMo
{
summary.registerException( e );
}
+ finally
+ {
+ cleanupForkConfiguration( forkConfiguration );
+ }
+ }
+
+ protected void cleanupForkConfiguration( ForkConfiguration
forkConfiguration )
+ {
+ if ( !getLog().isDebugEnabled() && forkConfiguration != null )
+ {
+ File tempDirectory = forkConfiguration.getTempDirectory();
+ try
+ {
+ FileUtils.deleteDirectory( tempDirectory );
+ }
+ catch ( IOException ioe )
+ {
+ getLog().warn( "Could not delete temp direcotry " +
tempDirectory + " because " + ioe.getMessage() );
+ }
+ }
}
protected abstract void handleSummary( Summary summary )
