Merge branch 'develop' into upgrade-sbt-scalafmt

sbt · Jul 6, 2024 · c24fc15 · c24fc15
2 parents ca84015 + e63b6cc
commit c24fc15
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 7 deletions.
diff --git a/core/src/main/scala/sbt/librarymanagement/ResolverExtra.scala b/core/src/main/scala/sbt/librarymanagement/ResolverExtra.scala
@@ -3,13 +3,15 @@
  */
 package sbt.librarymanagement
 
-import java.io.{ IOException, File }
+import java.io.{ File, IOException }
 import java.net.{ URI, URL }
 import scala.annotation.nowarn
 import scala.xml.XML
 import org.xml.sax.SAXParseException
 import sbt.util.Logger
+
 import java.net.URI
+import scala.util.matching.Regex
 
 final class RawRepository(val resolver: AnyRef, name: String) extends Resolver(name) {
   override def toString = "Raw(" + resolver.toString + ")"
@@ -400,20 +402,29 @@ private[librarymanagement] abstract class ResolverFunctions {
   def defaultRetrievePattern =
     "[type]s/[organisation]/[module]/" + PluginPattern + "[artifact](-[revision])(-[classifier]).[ext]"
   final val PluginPattern = "(scala_[scalaVersion]/)(sbt_[sbtVersion]/)"
-  private[librarymanagement] def expandMavenSettings(str: String): String = {
+
+  private[librarymanagement] def expandMavenSettings(
+      str: String,
+      envVars: Map[String, String] = sys.env,
+      props: Map[String, String] = sys.props.toMap
+  ): String = {
     // Aren't regular expressions beautifully clear and concise.
     // This means "find all ${...}" blocks, with the first group of each being the text between curly brackets.
-    val findQuoted = "\\$\\{([^\\}]*)\\}".r
+    val findQuoted = "\\$\\{([^}]*)}".r
     val env = "env\\.(.*)".r
 
     findQuoted.replaceAllIn(
       str,
-      _.group(1) match {
-        case env(variable) => sys.env.getOrElse(variable, "")
-        case property      => sys.props.getOrElse(property, "")
-      }
+      regexMatch =>
+        Regex.quoteReplacement {
+          regexMatch.group(1) match {
+            case env(variable) => envVars.getOrElse(variable, "")
+            case property      => props.getOrElse(property, "")
+          }
+        }
     )
   }
+
   private[this] def mavenLocalDir: File = {
     def loadHomeFromSettings(f: () => File): Option[File] =
       try {

diff --git a/core/src/test/scala/sbt/librarymanagement/ResolverExtraTest.scala b/core/src/test/scala/sbt/librarymanagement/ResolverExtraTest.scala
@@ -33,6 +33,13 @@ object ResolverExtraTest extends BasicTestSuite {
     )
   }
 
+  test("expandMavenSettings should preserve backslashes in environment variable values") {
+    val path = """C:\foo\bar\baz"""
+    val env = Map("SOME_PATH" -> path)
+
+    assert(Resolver.expandMavenSettings("${env.SOME_PATH}", env) == path)
+  }
+
   // - Helper functions ----------------------------------------------------------------------------
   // -----------------------------------------------------------------------------------------------
   def assertExpansion(input: String, expected: String) =