You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by ca...@apache.org on 2007/10/19 20:15:44 UTC
svn commit: r586566 -
/maven/components/trunk/maven-core/src/test/java/org/apache/maven/extension/DefaultExtensionManagerTest.java
Author: carlos
Date: Fri Oct 19 11:15:44 2007
New Revision: 586566
URL: http://svn.apache.org/viewvc?rev=586566&view=rev
Log:
Rolling back last workaround, files must not be locked, test is right and code needs to be fixed
Modified:
maven/components/trunk/maven-core/src/test/java/org/apache/maven/extension/DefaultExtensionManagerTest.java
Modified: maven/components/trunk/maven-core/src/test/java/org/apache/maven/extension/DefaultExtensionManagerTest.java
URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-core/src/test/java/org/apache/maven/extension/DefaultExtensionManagerTest.java?rev=586566&r1=586565&r2=586566&view=diff
==============================================================================
--- maven/components/trunk/maven-core/src/test/java/org/apache/maven/extension/DefaultExtensionManagerTest.java (original)
+++ maven/components/trunk/maven-core/src/test/java/org/apache/maven/extension/DefaultExtensionManagerTest.java Fri Oct 19 11:15:44 2007
@@ -83,17 +83,7 @@
if ( f.exists() )
{
- try
- {
- FileUtils.forceDelete( f );
- }
- catch ( IOException e )
- {
- // TODO carlos: delete fails on windows, needs investigation
- // the test-extension-1.jar is locked for deletion
- // I traced it and seems to get locked at getRealm().findRealmResources(
- // name ) in org.codehaus.plexus.classworlds.strategy.DefaultStrategy:146
- }
+ FileUtils.forceDelete( f );
}
}
}