Revert "Retrieving all licenses currently used in recipe jars"

This reverts commit 1696e91.

Author: jkschneider

rewrite-core/src/main/java/org/openrewrite/config/ClasspathScanningLoader.java

@@ -17,7 +17,6 @@
 
 import io.github.classgraph.ClassGraph;
 import io.github.classgraph.ClassInfo;
-import io.github.classgraph.Resource;
 import io.github.classgraph.ScanResult;
 import io.micrometer.core.instrument.Metrics;
 import io.micrometer.core.instrument.Timer;
@@ -30,9 +29,6 @@
 import org.openrewrite.internal.RecipeLoader;
 import org.openrewrite.style.NamedStyles;
 
-import java.io.BufferedReader;
-import java.io.IOException;
-import java.io.InputStreamReader;
 import java.lang.reflect.Constructor;
 import java.lang.reflect.Modifier;
 import java.nio.file.Path;
@@ -45,7 +41,6 @@ public class ClasspathScanningLoader implements ResourceLoader {
 
     private final LinkedHashMap<String, Recipe> recipes = new LinkedHashMap<>();
     private final List<NamedStyles> styles = new ArrayList<>();
-    private final Set<License> licenses = new LinkedHashSet<>();
 
     private final LinkedHashSet<RecipeDescriptor> recipeDescriptors = new LinkedHashSet<>();
     private final List<CategoryDescriptor> categoryDescriptors = new ArrayList<>();
@@ -70,7 +65,6 @@ public ClasspathScanningLoader(Properties properties, String[] acceptPackages) {
                     properties,
                     emptyList(),
                     null);
-            scanLicenses(new ClassGraph().acceptPathsNonRecursive("META-INF"));
         };
     }
 
@@ -94,7 +88,6 @@ public ClasspathScanningLoader(Properties properties, ClassLoader classLoader) {
                     properties,
                     emptyList(),
                     classLoader);
-            scanLicenses(new ClassGraph().acceptPathsNonRecursive("META-INF").overrideClassLoaders(classLoader));
         };
     }
 
@@ -113,15 +106,13 @@ public ClasspathScanningLoader(Path jar, Properties properties, Collection<? ext
                     .ignoreParentClassLoaders()
                     .overrideClassLoaders(classLoader)
                     .acceptPaths("META-INF/rewrite"), properties, dependencyResourceLoaders, classLoader);
-            scanLicenses(new ClassGraph().acceptPathsNonRecursive("META-INF").overrideClassLoaders(classLoader));
         };
     }
 
     public static ClasspathScanningLoader onlyYaml(Properties properties) {
         ClasspathScanningLoader classpathScanningLoader = new ClasspathScanningLoader();
         classpathScanningLoader.scanYaml(new ClassGraph().acceptPaths("META-INF/rewrite"),
                 properties, emptyList(), null);
-        classpathScanningLoader.scanLicenses(new ClassGraph().acceptPathsNonRecursive("META-INF"));
         return classpathScanningLoader;
     }
 
@@ -183,38 +174,6 @@ private void scanClasses(ClassGraph classGraph, ClassLoader classLoader) {
         }
     }
 
-    private void scanLicenses(ClassGraph classGraph) {
-        try (ScanResult result = classGraph.scan()) {
-            for (Resource resource : result.getResourcesWithLeafName("MANIFEST.MF")) {
-                try (Resource resToClose = resource; BufferedReader reader = new BufferedReader(new InputStreamReader(resource.open()))) {
-                    StringBuilder licenseName = new StringBuilder();
-                    StringBuilder licenseUrl = new StringBuilder();
-                    String previousLine = "";
-                    String line;
-                    while ((line = reader.readLine()) != null) {
-                        if (line.toLowerCase().startsWith("license-name:")) {
-                            licenseName = new StringBuilder(line.substring("license-name:".length()).trim());
-                        } else if (line.toLowerCase().startsWith("license-url:")) {
-                            licenseUrl = new StringBuilder(line.substring("license-url:".length()).trim());
-                        }
-                        if (line.startsWith(" ")) {
-                            if (previousLine.toLowerCase().startsWith("license-name:")) {
-                                licenseName.append(line.trim());
-                            } else if (previousLine.toLowerCase().startsWith("license-url:")) {
-                                licenseUrl.append(line.trim());
-                            }
-                        } else {
-                            previousLine = line;
-                        }
-                    }
-                    if (licenseName.length() > 0 && licenseUrl.length() > 0) {
-                        licenses.add(new License(licenseName.toString(), licenseUrl.toString()));
-                    }
-                } catch (IOException ignored) {}
-            }
-        }
-    }
-
     private void configureRecipes(ScanResult result, String className) {
         for (ClassInfo classInfo : result.getSubclasses(className)) {
             Class<?> recipeClass = classInfo.loadClass();
@@ -289,12 +248,6 @@ public Map<String, List<RecipeExample>> listRecipeExamples() {
         return recipeExamples;
     }
 
-    @Override
-    public Set<License> listLicenses() {
-        ensureScanned();
-        return licenses;
-    }
-
     @Override
     public Map<String, List<Contributor>> listContributors() {
         ensureScanned();

rewrite-core/src/main/java/org/openrewrite/config/Environment.java

@@ -29,7 +29,6 @@
 import java.nio.file.Path;
 import java.util.*;
 import java.util.function.Function;
-import java.util.stream.Collectors;
 
 import static java.util.Collections.emptyList;
 import static java.util.Comparator.comparingInt;
@@ -300,13 +299,6 @@ public List<NamedStyles> activateStyles(String... activeStyles) {
         return activateStyles(Arrays.asList(activeStyles));
     }
 
-    public Set<License> listLicenses() {
-        return resourceLoaders.stream()
-                .map(ResourceLoader::listLicenses)
-                .flatMap(Collection::stream)
-                .collect(Collectors.toSet());
-    }
-
     public Environment(Collection<? extends ResourceLoader> resourceLoaders) {
         this.resourceLoaders = resourceLoaders;
         this.dependencyResourceLoaders = emptyList();

rewrite-core/src/main/java/org/openrewrite/config/License.java

@@ -20,10 +20,7 @@
 import org.openrewrite.Recipe;
 import org.openrewrite.style.NamedStyles;
 
-import java.util.Collection;
-import java.util.List;
-import java.util.Map;
-import java.util.Set;
+import java.util.*;
 
 public interface ResourceLoader {
 
@@ -65,6 +62,4 @@ boolean includedIn(RecipeDetail... include) {
     Map<String, List<Contributor>> listContributors();
 
     Map<String, List<RecipeExample>> listRecipeExamples();
-
-    Set<License> listLicenses();
 }

rewrite-core/src/main/java/org/openrewrite/config/ResourceLoader.java

@@ -39,7 +39,6 @@
 import java.time.Duration;
 import java.util.*;
 import java.util.function.Consumer;
-import java.util.stream.Collectors;
 import java.util.stream.Stream;
 
 import static java.util.Collections.emptyList;
@@ -591,15 +590,6 @@ public Map<String, List<RecipeExample>> listRecipeExamples() {
         return recipeNameToExamples;
     }
 
-    @Override
-    public Set<License> listLicenses() {
-        return dependencyResourceLoaders.stream()
-                .filter(loader -> this != loader)
-                .map(ResourceLoader::listLicenses)
-                .flatMap(Collection::stream)
-                .collect(Collectors.toSet());
-    }
-
     @Override
     public Map<String, List<Contributor>> listContributors() {
         if (contributors == null) {