Author: brett
Date: Wed Sep 21 00:36:39 2005
New Revision: 290646

URL: http://svn.apache.org/viewcvs?rev=290646&view=rev
Log:
PR: MNG-917
don't overwrite dependency artifacts if they were already created

Modified:
    
maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java

Modified: 
maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java
URL: 
http://svn.apache.org/viewcvs/maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java?rev=290646&r1=290645&r2=290646&view=diff
==============================================================================
--- 
maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java
 (original)
+++ 
maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java
 Wed Sep 21 00:36:39 2005
@@ -1168,7 +1168,11 @@
         // check this with yourkit as a hot spot.
         try
         {
-            project.setDependencyArtifacts( project.createArtifacts( 
artifactFactory, null, null ) );
+            // Don't recreate if already created - for effeciency, and because 
clover plugin adds to it
+            if ( project.getDependencyArtifacts() == null )
+            {
+                project.setDependencyArtifacts( project.createArtifacts( 
artifactFactory, null, null ) );
+            }
         }
         catch ( InvalidVersionSpecificationException e )
         {



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to