You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@maven.apache.org by be...@apache.org on 2009/06/10 23:08:44 UTC

svn commit: r783520 - /maven/components/trunk/maven-core/src/main/java/org/apache/maven/lifecycle/DefaultLifecycleExecutor.java

Author: bentmann
Date: Wed Jun 10 21:08:44 2009
New Revision: 783520

URL: http://svn.apache.org/viewvc?rev=783520&view=rev
Log:
[MNG-3401] Plugin parameters must be specified outside an execution block when they are invoked from the command line

Modified:
    maven/components/trunk/maven-core/src/main/java/org/apache/maven/lifecycle/DefaultLifecycleExecutor.java

Modified: maven/components/trunk/maven-core/src/main/java/org/apache/maven/lifecycle/DefaultLifecycleExecutor.java
URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-core/src/main/java/org/apache/maven/lifecycle/DefaultLifecycleExecutor.java?rev=783520&r1=783519&r2=783520&view=diff
==============================================================================
--- maven/components/trunk/maven-core/src/main/java/org/apache/maven/lifecycle/DefaultLifecycleExecutor.java (original)
+++ maven/components/trunk/maven-core/src/main/java/org/apache/maven/lifecycle/DefaultLifecycleExecutor.java Wed Jun 10 21:08:44 2009
@@ -304,7 +304,7 @@
         
         MojoDescriptor mojoDescriptor = getMojoDescriptor( goal, session );
 
-        MojoExecution mojoExecution = new MojoExecution( mojoDescriptor, "default-" + mojoDescriptor.getGoal() );
+        MojoExecution mojoExecution = new MojoExecution( mojoDescriptor, "default-cli" );
         
         populateMojoExecutionConfiguration( project, mojoExecution, true );
 
@@ -839,7 +839,6 @@
             String[] p = StringUtils.split( goal, ":" );
 
             PluginExecution execution = new PluginExecution();
-            // FIXME: Find a better execution id
             execution.setId( "default-" + p[2] );
             execution.setPhase( phase );
             execution.getGoals().add( p[2] );