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

Posted to commits@maven.apache.org by rf...@apache.org on 2012/10/27 23:37:30 UTC

svn commit: r1402891 - /maven/plugins/trunk/maven-invoker-plugin/src/main/java/org/apache/maven/plugin/invoker/AbstractInvokerMojo.java

Author: rfscholte
Date: Sat Oct 27 21:37:30 2012
New Revision: 1402891

URL: http://svn.apache.org/viewvc?rev=1402891&view=rev
Log:
Add reminder. Related to MINVOKER-126: mavenHome in relationship with invoker.maven.version rule 

Modified:
    maven/plugins/trunk/maven-invoker-plugin/src/main/java/org/apache/maven/plugin/invoker/AbstractInvokerMojo.java

Modified: maven/plugins/trunk/maven-invoker-plugin/src/main/java/org/apache/maven/plugin/invoker/AbstractInvokerMojo.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-invoker-plugin/src/main/java/org/apache/maven/plugin/invoker/AbstractInvokerMojo.java?rev=1402891&r1=1402890&r2=1402891&view=diff
==============================================================================
--- maven/plugins/trunk/maven-invoker-plugin/src/main/java/org/apache/maven/plugin/invoker/AbstractInvokerMojo.java (original)
+++ maven/plugins/trunk/maven-invoker-plugin/src/main/java/org/apache/maven/plugin/invoker/AbstractInvokerMojo.java Sat Oct 27 21:37:30 2012
@@ -1102,6 +1102,9 @@ public abstract class AbstractInvokerMoj
             }
         }
         final File finalSettingsFile = mergedSettingsFile;
+        
+        // @todo calculate MavenVersion, JavaVersion and OS here only once...
+        // isSelected() retrieves these values for every BuildJob, but they are read over and over again inside SelectorUtils 
 
         try
         {