sort revisions semantically

fixes #3925

Author: vladak

opengrok-indexer/pom.xml

@@ -18,7 +18,7 @@ information: Portions Copyright [yyyy] [name of copyright owner]
 
 CDDL HEADER END
 
-Copyright (c) 2010, 2021, Oracle and/or its affiliates. All rights reserved.
+Copyright (c) 2010, 2022, Oracle and/or its affiliates. All rights reserved.
 Portions Copyright (c) 2017-2020, Chris Fraire <[email protected]>.
 Portions Copyright (c) 2020-2020, Lubos Kosco <[email protected]>.
 
@@ -223,6 +223,11 @@ Portions Copyright (c) 2020-2020, Lubos Kosco <[email protected]>.
             <artifactId>org.eclipse.jgit</artifactId>
             <version>5.12.0.202106070339-r</version>
         </dependency>
+        <dependency>
+            <groupId>io.github.g00fy2</groupId>
+            <artifactId>versioncompare</artifactId>
+            <version>1.5.0</version>
+        </dependency>
     </dependencies>
 
     <build>

opengrok-indexer/src/main/java/org/opengrok/indexer/history/CVSHistoryParser.java

@@ -18,7 +18,7 @@
  */
 
 /*
- * Copyright (c) 2008, 2021, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2008, 2022, Oracle and/or its affiliates. All rights reserved.
  */
 package org.opengrok.indexer.history;
 
@@ -33,6 +33,8 @@
 import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.List;
+
+import io.github.g00fy2.versioncompare.Version;
 import org.opengrok.indexer.util.Executor;
 
 /**
@@ -182,7 +184,7 @@ History parse(File file, Repository repos) throws HistoryException {
         // so they need to be sorted according to revision.
         if (cvsrepo.getBranch() != null && !cvsrepo.getBranch().isEmpty()) {
             List<HistoryEntry> entries = history.getHistoryEntries();
-            entries.sort((o1, o2) -> o2.getRevision().compareTo(o1.getRevision()));
+            entries.sort((o1, o2) -> new Version(o2.getRevision()).compareTo(new Version(o1.getRevision())));
             history.setHistoryEntries(entries);
         }