From 861b8aa7921090f3abf1106d432181812d5fd6d6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jo=C3=A3o=20Ferreira?= <jtjeferreira@gmail.com>
Date: Wed, 29 May 2024 17:49:36 +0100
Subject: [PATCH] remove a few more java.net.URL

---
 .../librarymanagement/ComponentManager.scala         |  5 +----
 .../internal/librarymanagement/ConvertResolver.scala | 12 ++++++------
 .../internal/librarymanagement/FakeResolver.scala    |  2 +-
 3 files changed, 8 insertions(+), 11 deletions(-)

diff --git a/ivy/src/main/scala/sbt/internal/librarymanagement/ComponentManager.scala b/ivy/src/main/scala/sbt/internal/librarymanagement/ComponentManager.scala
index 710c120de..7f84c3bf8 100644
--- a/ivy/src/main/scala/sbt/internal/librarymanagement/ComponentManager.scala
+++ b/ivy/src/main/scala/sbt/internal/librarymanagement/ComponentManager.scala
@@ -101,7 +101,7 @@ object IfMissing {
 object ComponentManager {
   lazy val (version, timestamp) = {
     val properties = new java.util.Properties
-    val propertiesStream = versionResource.openStream
+    val propertiesStream = getClass.getResourceAsStream("/xsbt.version.properties")
     try {
       properties.load(propertiesStream)
     } finally {
@@ -110,7 +110,4 @@ object ComponentManager {
     (properties.getProperty("version"), properties.getProperty("timestamp"))
   }
   lazy val stampedVersion = version + "_" + timestamp
-
-  import java.net.URL
-  private def versionResource: URL = getClass.getResource("/xsbt.version.properties")
 }
diff --git a/ivy/src/main/scala/sbt/internal/librarymanagement/ConvertResolver.scala b/ivy/src/main/scala/sbt/internal/librarymanagement/ConvertResolver.scala
index 1bd0d7aad..ef59c2e02 100644
--- a/ivy/src/main/scala/sbt/internal/librarymanagement/ConvertResolver.scala
+++ b/ivy/src/main/scala/sbt/internal/librarymanagement/ConvertResolver.scala
@@ -394,17 +394,17 @@ private[sbt] object ConvertResolver {
     private[this] val repo = new WarnOnOverwriteFileRepo()
     private[this] val progress = new RepositoryCopyProgressListener(this);
     override def getResource(source: String) = {
-      val url = new URI(source).toURL
-      if (url.getProtocol == IO.FileScheme)
-        new FileResource(repo, IO.toFile(url))
+      val uri = new URI(source)
+      if (uri.getScheme == IO.FileScheme)
+        new FileResource(repo, IO.toFile(uri))
       else
         super.getResource(source)
     }
 
     override def put(source: File, destination: String, overwrite: Boolean): Unit = {
-      val url = new URI(destination).toURL
+      val uri = new URI(destination)
       try {
-        if (url.getProtocol != IO.FileScheme) super.put(source, destination, overwrite)
+        if (uri.getScheme != IO.FileScheme) super.put(source, destination, overwrite)
         else {
           // Here we duplicate the put method for files so we don't just bail on trying ot use Http handler
           val resource = getResource(destination)
@@ -417,7 +417,7 @@ private[sbt] object ConvertResolver {
             if (totalLength > 0) {
               progress.setTotalLength(totalLength);
             }
-            FileUtil.copy(source, new java.io.File(url.toURI), progress, overwrite)
+            FileUtil.copy(source, new java.io.File(uri), progress, overwrite)
             ()
           } catch {
             case ex: IOException =>
diff --git a/ivy/src/main/scala/sbt/internal/librarymanagement/FakeResolver.scala b/ivy/src/main/scala/sbt/internal/librarymanagement/FakeResolver.scala
index 2d6ef89de..00c0275ae 100644
--- a/ivy/src/main/scala/sbt/internal/librarymanagement/FakeResolver.scala
+++ b/ivy/src/main/scala/sbt/internal/librarymanagement/FakeResolver.scala
@@ -69,7 +69,7 @@ private[sbt] class FakeResolver(private var name: String, cacheDir: File, module
   ): ArtifactDownloadReport = {
 
     val report = new ArtifactDownloadReport(artifact.getArtifact)
-    val path = new URI(artifact.getLocation).toURL.toURI.getPath
+    val path = new URI(artifact.getLocation).getPath
     val localFile = new File(path)
 
     if (path.nonEmpty && localFile.exists) {