svn commit: r656821 - /maven/surefire/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java

[hb...@apache.org]

Author: hboutemy
Date: Thu May 15 13:34:14 2008
New Revision: 656821

URL: http://svn.apache.org/viewvc?rev=656821&view=rev
Log:
updated comments

Modified:
    maven/surefire/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java

Modified: maven/surefire/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java
URL: http://svn.apache.org/viewvc/maven/surefire/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java?rev=656821&r1=656820&r2=656821&view=diff
==============================================================================
--- maven/surefire/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java (original)
+++ maven/surefire/trunk/maven-surefire-plugin/src/main/java/org/apache/maven/plugin/surefire/SurefirePlugin.java Thu May 15 13:34:14 2008
@@ -645,8 +645,7 @@
             throw new MojoExecutionException( "Unable to locate surefire-booter in the list of plugin artifacts" );
         }
 
-        surefireArtifact.isSnapshot(); // TODO: this is ridiculous, but it fixes getBaseVersion to be -SNAPSHOT if
-        // needed
+        surefireArtifact.isSnapshot(); // MNG-2961: before Maven 2.0.8, fixes getBaseVersion to be -SNAPSHOT if needed
 
         Artifact junitArtifact;
         Artifact testNgArtifact;
@@ -1124,7 +1123,7 @@
     }
     
     //TODO remove the part with ToolchainManager lookup once we depend on
-    //3.0.9 (have it as prerequisite). Define as regular component field then.
+    //2.0.9 (have it as prerequisite). Define as regular component field then.
     private Toolchain getToolchain() 
     {
         Toolchain tc = null;
@@ -1139,7 +1138,7 @@
                 }
             }
         } catch (ComponentLookupException componentLookupException) {
-            //just ignore, could happen in pre-3.0.9 builds..
+            //just ignore, could happen in pre-2.0.9 builds..
         }
         return tc;
     }