Return-Path: Delivered-To: apmail-maven-commits-archive@www.apache.org Received: (qmail 94297 invoked from network); 7 Sep 2006 14:25:49 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur.apache.org with SMTP; 7 Sep 2006 14:25:49 -0000 Received: (qmail 43040 invoked by uid 500); 7 Sep 2006 14:25:48 -0000 Delivered-To: apmail-maven-commits-archive@maven.apache.org Received: (qmail 42797 invoked by uid 500); 7 Sep 2006 14:25:48 -0000 Mailing-List: contact commits-help@maven.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@maven.apache.org Delivered-To: mailing list commits@maven.apache.org Received: (qmail 42786 invoked by uid 99); 7 Sep 2006 14:25:48 -0000 Received: from asf.osuosl.org (HELO asf.osuosl.org) (140.211.166.49) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 07 Sep 2006 07:25:48 -0700 X-ASF-Spam-Status: No, hits=-9.4 required=10.0 tests=ALL_TRUSTED,NO_REAL_NAME X-Spam-Check-By: apache.org Received-SPF: pass (asf.osuosl.org: local policy) Received: from [140.211.166.113] (HELO eris.apache.org) (140.211.166.113) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 07 Sep 2006 07:25:47 -0700 Received: by eris.apache.org (Postfix, from userid 65534) id BFD3B1A981A; Thu, 7 Sep 2006 07:25:27 -0700 (PDT) Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: svn commit: r441093 - /maven/plugins/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java Date: Thu, 07 Sep 2006 14:25:27 -0000 To: commits@maven.apache.org From: kenney@apache.org X-Mailer: svnmailer-1.1.0 Message-Id: <20060907142527.BFD3B1A981A@eris.apache.org> X-Virus-Checked: Checked by ClamAV on apache.org X-Spam-Rating: minotaur.apache.org 1.6.2 0/1000/N Author: kenney Date: Thu Sep 7 07:25:27 2006 New Revision: 441093 URL: http://svn.apache.org/viewvc?view=rev&rev=441093 Log: PR: SUREFIRE-54 Related to commit revision 441092. Removed adding test classpathelements to the surefire classpath since the surefire classloader is a child of the test classloader and already has access to all the test classes. 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/viewvc/maven/plugins/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java?view=diff&rev=441093&r1=441092&r2=441093 ============================================================================== --- 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 Thu Sep 7 07:25:27 2006 @@ -585,11 +585,6 @@ getLog().debug( " " + classpathElement ); surefireBooter.addClassPathUrl( classpathElement ); - - // TODO: fix - // Some test suites (TestNG) do not utilise the separate classloader, so we need to make all the tests - // available in the main classloader as well (this is consistent with the old behaviour in Surefire 1.5) - surefireBooter.addSurefireClassPathUrl( classpathElement ); } // ----------------------------------------------------------------------