You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tuscany.apache.org by rf...@apache.org on 2009/04/16 02:49:40 UTC

svn commit: r765438 - /tuscany/maven-plugins/trunk/maven-osgi-junit/src/main/java/org/apache/tuscany/maven/plugin/surefire/OSGiSurefireBooter.java

Author: rfeng
Date: Thu Apr 16 00:49:40 2009
New Revision: 765438

URL: http://svn.apache.org/viewvc?rev=765438&view=rev
Log:
Remove appliation classpath from poluting the forked tests in OSGi

Modified:
    tuscany/maven-plugins/trunk/maven-osgi-junit/src/main/java/org/apache/tuscany/maven/plugin/surefire/OSGiSurefireBooter.java

Modified: tuscany/maven-plugins/trunk/maven-osgi-junit/src/main/java/org/apache/tuscany/maven/plugin/surefire/OSGiSurefireBooter.java
URL: http://svn.apache.org/viewvc/tuscany/maven-plugins/trunk/maven-osgi-junit/src/main/java/org/apache/tuscany/maven/plugin/surefire/OSGiSurefireBooter.java?rev=765438&r1=765437&r2=765438&view=diff
==============================================================================
--- tuscany/maven-plugins/trunk/maven-osgi-junit/src/main/java/org/apache/tuscany/maven/plugin/surefire/OSGiSurefireBooter.java (original)
+++ tuscany/maven-plugins/trunk/maven-osgi-junit/src/main/java/org/apache/tuscany/maven/plugin/surefire/OSGiSurefireBooter.java Thu Apr 16 00:49:40 2009
@@ -573,13 +573,18 @@
         System.out.println("sbcp: " + surefireBootClassPathUrls);
         */
         
-        List bootClasspath = new ArrayList(surefireBootClassPathUrls.size() + classPathUrls.size());
+        List bootClasspath = new ArrayList(surefireBootClassPathUrls.size());
 
         bootClasspath.addAll(surefireBootClassPathUrls);
 
+        /**
+         * For OSGi, we don't want to polute the system classpath 
+         */
+        /*
         if (useSystemClassLoader()) {
             bootClasspath.addAll(classPathUrls);
         }
+        */
 
         Commandline cli =
             forkConfiguration.createCommandLine(bootClasspath, useManifestOnlyJar());