fix(analyze-command): Properly close PackageManagers after they have …

…retrieved all dependencies.

Signed-off-by: Thomas Neidhart <[email protected]>

analyzer/src/main/kotlin/PackageManager.kt

@@ -19,6 +19,7 @@
 
 package org.ossreviewtoolkit.analyzer
 
+import java.io.Closeable
 import java.io.File
 import java.nio.file.FileSystems
 import java.nio.file.Path
@@ -62,7 +63,7 @@ abstract class PackageManager(
     val analysisRoot: File,
     val analyzerConfig: AnalyzerConfiguration,
     val repoConfig: RepositoryConfiguration
-) {
+) : Closeable {
     companion object {
         private val PACKAGE_MANAGER_DIRECTORIES = setOf(
             // Ignore intermediate build system directories.
@@ -361,6 +362,11 @@ abstract class PackageManager(
             }
         }
     }
+
+    /**
+     * [PackageManager] implementations should override this method if they need to clean resources.
+     */
+    override fun close() {}
 }
 
 /**

plugins/commands/analyzer/src/main/kotlin/AnalyzerCommand.kt

@@ -228,5 +228,9 @@ class AnalyzerCommand : OrtCommand(
         val issues = analyzerRun.result.getAllIssues().flatMap { it.value }
         SeverityStatsPrinter(terminal, resolutionProvider).stats(issues)
             .print().conclude(ortConfig.severeIssueThreshold, 2)
+
+        for (packageManager in info.managedFiles.keys) {
+            packageManager.close()
+        }
     }
 }

plugins/package-managers/gradle/src/main/kotlin/Gradle.kt

@@ -290,4 +290,8 @@ class Gradle(
             }
         }
     }
+
+    override fun close() {
+        maven.close()
+    }
 }

plugins/package-managers/maven/src/main/kotlin/Maven.kt

@@ -65,6 +65,10 @@ class Maven(
         ) = Maven(type, analysisRoot, analyzerConfig, repoConfig)
     }
 
+    override fun close() {
+        mvn.close()
+    }
+
     private inner class LocalProjectWorkspaceReader : WorkspaceReader {
         private val workspaceRepository = WorkspaceRepository()
 

plugins/package-managers/maven/src/main/kotlin/utils/MavenSupport.kt

@@ -19,6 +19,7 @@
 
 package org.ossreviewtoolkit.plugins.packagemanagers.maven.utils
 
+import java.io.Closeable
 import java.io.File
 import java.net.URI
 
@@ -108,7 +109,7 @@ fun Artifact.identifier() = "$groupId:$artifactId:$version"
 private val File?.safePath: String
     get() = this?.invariantSeparatorsPath ?: "<unknown file>"
 
-class MavenSupport(private val workspaceReader: WorkspaceReader) {
+class MavenSupport(private val workspaceReader: WorkspaceReader) : Closeable {
     companion object {
         private val PACKAGING_TYPES = setOf(
             // Core packaging types, see https://maven.apache.org/pom.html#packaging.
@@ -814,6 +815,10 @@ class MavenSupport(private val workspaceReader: WorkspaceReader) {
             legacySupport.session = null
         }
     }
+
+    override fun close() {
+        remoteArtifactCache.close()
+    }
 }
 
 /**

utils/common/src/main/kotlin/DiskCache.kt

@@ -21,6 +21,7 @@ package org.ossreviewtoolkit.utils.common
 
 import com.jakewharton.disklrucache.DiskLruCache
 
+import java.io.Closeable
 import java.io.File
 import java.io.IOException
 
@@ -46,7 +47,7 @@ class DiskCache(
      * Duration in seconds that cache entries are valid.
      */
     private val maxCacheEntryAgeInSeconds: Long
-) {
+) : Closeable {
     companion object {
         const val INDEX_FULL_KEY = 0
         const val INDEX_TIMESTAMP = 1
@@ -144,4 +145,8 @@ class DiskCache(
     }
 
     private fun currentTimeInSeconds() = System.currentTimeMillis() / 1000L
+
+    override fun close() {
+        diskLruCache.close()
+    }
 }