Replace float comparision with String comparison, otherwise the floats 4.1.x and...
authorRobert Scholte <rfscholte@apache.org>
Sun, 12 Jun 2016 10:15:49 +0000 (10:15 +0000)
committerRobert Scholte <rfscholte@apache.org>
Sun, 12 Jun 2016 10:15:49 +0000 (10:15 +0000)
git-svn-id: https://svn.apache.org/repos/asf/maven/shared/trunk@1747973 13f79535-47bb-0310-9956-ffa450edef68

src/main/java/org/apache/maven/shared/project/runtime/MavenUtils.java
src/main/java/org/apache/maven/shared/project/utils/AnsiUtils.java
src/test/java/org/apache/maven/shared/project/runtime/MavenUtilsTest.java [new file with mode: 0644]

index 450b026..3e0f93a 100644 (file)
@@ -23,6 +23,7 @@ import java.io.IOException;
 import java.io.InputStream;
 import java.util.Properties;
 
+import org.apache.maven.artifact.versioning.DefaultArtifactVersion;
 import org.apache.maven.project.MavenProject;
 import org.codehaus.plexus.util.IOUtil;
 
@@ -63,13 +64,15 @@ public final class MavenUtils
     }
 
     /**
-     * Get Maven version as major.minor float
+     * Returns a positive value if the version parameter is bigger compared to the runtime Maven version
+     * Returns a negative value if the version parameter is less compared to the runtime Maven version
+     * Returns 0 if they are the same.
      * 
-     * @return
+     * @param version the version to compare
+     * @return 
      */
-    public static float getMavenVersionAsFloat()
+    public static int compareToVersion( String version )
     {
-        String[] digits = getMavenVersion().split( "\\." );
-        return Float.parseFloat( digits[0] + '.' + digits[1] );
+        return new DefaultArtifactVersion( version ).compareTo( new DefaultArtifactVersion( getMavenVersion() ) );
     }
 }
index fea225b..5fa4401 100644 (file)
@@ -28,7 +28,7 @@ import org.fusesource.jansi.AnsiConsole;
  */
 public class AnsiUtils
 {
-    private static final float MINIMUM_MAVEN_VERSION = 3.4f; // color added in Maven 3.4.0: see MNG-3507
+    private static final String MINIMUM_MAVEN_VERSION = "3.4.0"; // color added in Maven 3.4.0: see MNG-3507
 
     private AnsiUtils()
     {
@@ -37,7 +37,7 @@ public class AnsiUtils
     public static void systemInstall()
     {
         AnsiConsole.systemInstall();
-        if ( MavenUtils.getMavenVersionAsFloat() < MINIMUM_MAVEN_VERSION )
+        if ( MavenUtils.compareToVersion( MINIMUM_MAVEN_VERSION ) >= 0 )
         {
             // ANSI color support was added in Maven 3.4.0: don't enable color if executing older Maven versions
             Ansi.setEnabled( false );
diff --git a/src/test/java/org/apache/maven/shared/project/runtime/MavenUtilsTest.java b/src/test/java/org/apache/maven/shared/project/runtime/MavenUtilsTest.java
new file mode 100644 (file)
index 0000000..ab73561
--- /dev/null
@@ -0,0 +1,36 @@
+package org.apache.maven.shared.project.runtime;\r
+\r
+/*\r
+ * Licensed to the Apache Software Foundation (ASF) under one\r
+ * or more contributor license agreements.  See the NOTICE file\r
+ * distributed with this work for additional information\r
+ * regarding copyright ownership.  The ASF licenses this file\r
+ * to you under the Apache License, Version 2.0 (the\r
+ * "License"); you may not use this file except in compliance\r
+ * with the License.  You may obtain a copy of the License at\r
+ *\r
+ *  http://www.apache.org/licenses/LICENSE-2.0\r
+ *\r
+ * Unless required by applicable law or agreed to in writing,\r
+ * software distributed under the License is distributed on an\r
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY\r
+ * KIND, either express or implied.  See the License for the\r
+ * specific language governing permissions and limitations\r
+ * under the License.\r
+ */\r
+\r
+import static org.junit.Assert.assertTrue;\r
+\r
+import org.junit.Test;\r
+\r
+public class MavenUtilsTest\r
+{\r
+\r
+    @Test\r
+    public void testCompareToVersion()\r
+    {\r
+        assertTrue( MavenUtils.compareToVersion( "101.0.0" ) > 0 );\r
+        assertTrue( MavenUtils.compareToVersion( MavenUtils.getMavenVersion() ) == 0 );\r
+        assertTrue( MavenUtils.compareToVersion( "0.0.1" ) < 0 );\r
+    }\r
+}\r