Re: svn commit: r730028 - /maven/components/trunk/maven-model/src/main/mdo/maven.mdo

[Benjamin Bentmann]

Author: sisbell
Date: Mon Dec 29 17:33:12 2008
New Revision: 730028
URL: http://svn.apache.org/viewvc?rev=730028&view=rev
Log:
Fix: MNG-3927

Modified:
    maven/components/trunk/maven-model/src/main/mdo/maven.mdo

Modified: maven/components/trunk/maven-model/src/main/mdo/maven.mdo
URL: 
http://svn.apache.org/viewvc/maven/components/trunk/maven-model/src/main/mdo/maven.mdo?rev=730028&r1=730027&r2=730028&view=diff
==============================================================================
--- maven/components/trunk/maven-model/src/main/mdo/maven.mdo (original)
+++ maven/components/trunk/maven-model/src/main/mdo/maven.mdo Mon Dec 29 
17:33:12 2008
@@ -2733,7 +2733,7 @@
           <name>id</name>
           <version>4.0.0</version>
           <type>String</type>
-          <defaultValue>default</defaultValue>
+          <defaultValue>default-execution-id</defaultValue>
           <description>The identifier of this execution for labelling the 
goals during the build,
             and for matching executions to merge during 
inheritance.</description>
         </field>
@@ -2759,7 +2759,7 @@
           <version>4.0.0</version>
           <code>
             <![CDATA[
-    public static final String DEFAULT_EXECUTION_ID = "default";
+    public static final String DEFAULT_EXECUTION_ID = "default-execution-id";

Hm, looks odd that the fix for a regression is to break backward-compat 
somewhere else. Was that really the best option available? What if we 
change the default value for the field "executionId" in 
o.a.m.l.m.MojoBinding from maven-lifecycle instead?


Benjamin

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org