You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by br...@apache.org on 2006/05/01 14:43:48 UTC
svn commit: r398583 -
/maven/plugins/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java
Author: brett
Date: Mon May 1 05:43:47 2006
New Revision: 398583
URL: http://svn.apache.org/viewcvs?rev=398583&view=rev
Log:
[MSUREFIRE-74] avoid doubling up on the classpaths listed (though it was inconsequential)
Modified:
maven/plugins/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java
Modified: maven/plugins/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java
URL: http://svn.apache.org/viewcvs/maven/plugins/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java?rev=398583&r1=398582&r2=398583&view=diff
==============================================================================
--- maven/plugins/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java (original)
+++ maven/plugins/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java Mon May 1 05:43:47 2006
@@ -543,13 +543,7 @@
getLog().debug( "Test Classpath :" );
- getLog().debug( " " + testClassesDirectory.getPath() );
-
- surefireBooter.addClassPathUrl( testClassesDirectory.getPath() );
-
- getLog().debug( " " + classesDirectory.getPath() );
-
- surefireBooter.addClassPathUrl( classesDirectory.getPath() );
+ // no need to add classes/test classes directory here - they are in the classpath elements already
for ( Iterator i = classpathElements.iterator(); i.hasNext(); )
{