You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@maven.apache.org by br...@apache.org on 2005/09/21 09:36:43 UTC
svn commit: r290646 -
/maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java
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: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org