Revert "Enable ABI checking on modules with public APIs"

1 files changed, 4 insertions, 13 deletions

diff --git a/abi-check-plugin/src/main/java/com/yahoo/abicheck/mojo/AbiCheck.java b/abi-check-plugin/src/main/java/com/yahoo/abicheck/mojo/AbiCheck.java
index 2ee7bd495b3..3ae1102014d 100644
--- a/abi-check-plugin/src/main/java/com/yahoo/abicheck/mojo/AbiCheck.java
+++ b/abi-check-plugin/src/main/java/com/yahoo/abicheck/mojo/AbiCheck.java
@@ -1,6 +1,5 @@
 package com.yahoo.abicheck.mojo;
 
-import com.google.common.collect.Ordering;
 import com.google.gson.Gson;
 import com.google.gson.GsonBuilder;
 import com.google.gson.reflect.TypeToken;
@@ -16,14 +15,11 @@ import java.io.FileReader;
 import java.io.FileWriter;
 import java.io.IOException;
 import java.io.InputStream;
-import java.util.Comparator;
 import java.util.HashSet;
 import java.util.LinkedHashMap;
-import java.util.List;
 import java.util.Map;
 import java.util.Optional;
 import java.util.jar.JarFile;
-import java.util.stream.Collectors;
 import org.apache.maven.artifact.Artifact;
 import org.apache.maven.plugin.AbstractMojo;
 import org.apache.maven.plugin.MojoExecutionException;
@@ -72,7 +68,7 @@ public class AbiCheck extends AbstractMojo {
   // Testing that Gson can write JSON files is not very useful
   private static void writeSpec(Map<String, JavaClassSignature> signatures, File file)
       throws IOException {
-    Gson gson = new GsonBuilder().setPrettyPrinting().disableHtmlEscaping().create();
+    Gson gson = new GsonBuilder().setPrettyPrinting().create();
     try (FileWriter writer = new FileWriter(file)) {
       gson.toJson(signatures, writer);
     }
@@ -136,19 +132,14 @@ public class AbiCheck extends AbstractMojo {
     Map<String, JavaClassSignature> signatures = new LinkedHashMap<>();
     if (isPublicAbiPackage(pkg, publicApiAnnotation)) {
       PublicSignatureCollector collector = new PublicSignatureCollector();
-      List<ClassFileTree.ClassFile> sortedClassFiles = pkg.getClassFiles().stream()
-          .sorted(Comparator.comparing(ClassFile::getName)).collect(Collectors.toList());
-      for (ClassFile klazz : sortedClassFiles) {
+      for (ClassFile klazz : pkg.getClassFiles()) {
         try (InputStream is = klazz.getInputStream()) {
           new ClassReader(is).accept(collector, 0);
         }
       }
       signatures.putAll(collector.getClassSignatures());
     }
-    List<ClassFileTree.Package> sortedSubPackages = pkg.getSubPackages().stream()
-        .sorted(Comparator.comparing(Package::getFullyQualifiedName))
-        .collect(Collectors.toList());
-    for (ClassFileTree.Package subPkg : sortedSubPackages) {
+    for (ClassFileTree.Package subPkg : pkg.getSubPackages()) {
       signatures.putAll(collectPublicAbiSignatures(subPkg, publicApiAnnotation));
     }
     return signatures;
@@ -170,7 +161,7 @@ public class AbiCheck extends AbstractMojo {
     File specFile = new File(project.getBasedir(), specFileName);
     if (mainArtifact.getFile() == null) {
       throw new MojoExecutionException("Missing project artifact file");
-    } else if (!mainArtifact.getFile().getName().endsWith(".jar")) {
+    } else if (!mainArtifact.getType().equals("jar")) {
       throw new MojoExecutionException("Project artifact is not a JAR");
     }