Merge pull request #1630 from lwronski/scala-version-repo

Add repositories from build options when validating scala versions

Author: lwronski

modules/build/src/main/scala/scala/build/LocalRepo.scala

@@ -1,15 +1,17 @@
 package scala.build
 
+import coursier.core.{Repository, Version}
+import coursier.parse.RepositoryParser
 import coursier.paths.Util
 
 import java.io.{BufferedInputStream, Closeable}
 import java.nio.channels.{FileChannel, FileLock}
 import java.nio.charset.StandardCharsets
 import java.nio.file.{Path, StandardOpenOption}
 
+import scala.build.errors.{BuildException, RepositoryFormatError}
 import scala.build.internal.Constants
 import scala.build.internal.zip.WrappedZipInputStream
-
 object LocalRepo {
 
   private def resourcePath = Constants.localRepoResourcePath

modules/build/src/main/scala/scala/build/ReplArtifacts.scala

@@ -1,6 +1,7 @@
 package scala.build
 
 import coursier.cache.FileCache
+import coursier.core.{Repository, Version}
 import coursier.util.Task
 import dependency._
 
@@ -42,25 +43,25 @@ object ReplArtifacts {
     dependencies: Seq[AnyDependency],
     extraClassPath: Seq[os.Path],
     extraSourceJars: Seq[os.Path],
+    extraRepositories: Seq[Repository],
     logger: Logger,
     cache: FileCache[Task],
     directories: Directories,
     addScalapy: Option[String]
   ): Either[BuildException, ReplArtifacts] = either {
-    val localRepoOpt = LocalRepo.localRepo(directories.localRepoDir)
     val scalapyDeps =
       addScalapy.map(ver => dep"${Artifacts.scalaPyOrganization(ver)}::scalapy-core::$ver").toSeq
     val allDeps = dependencies ++ Seq(dep"com.lihaoyi:::ammonite:$ammoniteVersion") ++ scalapyDeps
     val replArtifacts = Artifacts.artifacts(
       Positioned.none(allDeps),
-      localRepoOpt.toSeq,
+      extraRepositories,
       Some(scalaParams),
       logger,
       cache.withMessage(s"Downloading Ammonite $ammoniteVersion")
     )
     val replSourceArtifacts = Artifacts.artifacts(
       Positioned.none(allDeps),
-      localRepoOpt.toSeq,
+      extraRepositories,
       Some(scalaParams),
       logger,
       cache.withMessage(s"Downloading Ammonite $ammoniteVersion sources"),
@@ -82,7 +83,7 @@ object ReplArtifacts {
     extraClassPath: Seq[os.Path],
     logger: Logger,
     cache: FileCache[Task],
-    repositories: Seq[String],
+    repositories: Seq[Repository],
     addScalapy: Option[String]
   ): Either[BuildException, ReplArtifacts] = either {
     val isScala2 = scalaParams.scalaVersion.startsWith("2.")

modules/build/src/test/scala/scala/build/tests/ReplArtifactsTests.scala

@@ -16,6 +16,7 @@ class ReplArtifactsTests extends munit.FunSuite {
         Nil,
         Nil,
         Nil,
+        Nil,
         Logger.nop,
         FileCache(),
         Directories.under(root),

modules/cli/src/main/scala/scala/cli/commands/ScalaCommand.scala

@@ -7,6 +7,7 @@ import caseapp.core.help.{Help, HelpFormat}
 import caseapp.core.parser.Parser
 import caseapp.core.util.Formatter
 import caseapp.core.{Arg, Error, RemainingArgs}
+import coursier.core.{Repository, Version}
 import dependency.*
 
 import scala.annotation.tailrec
@@ -180,7 +181,7 @@ abstract class ScalaCommand[T <: HasLoggingOptions](implicit myParser: Parser[T]
             val docArtifacts = value {
               Artifacts.fetch(
                 Positioned.none(Seq(dep"org.scala-lang::scaladoc:${scalaParams.scalaVersion}")),
-                buildOptions.finalRepositories,
+                value(buildOptions.finalRepositories),
                 Some(scalaParams),
                 logger,
                 buildOptions.finalCache,
@@ -211,7 +212,7 @@ abstract class ScalaCommand[T <: HasLoggingOptions](implicit myParser: Parser[T]
                 Positioned.none(Seq(
                   dep"${Constants.scalafmtOrganization}:${Constants.scalafmtName}:${Constants.defaultScalafmtVersion}"
                 )),
-                buildOptions.finalRepositories,
+                value(buildOptions.finalRepositories),
                 Some(scalaParams),
                 logger,
                 buildOptions.finalCache,

modules/cli/src/main/scala/scala/cli/commands/doc/Doc.scala

@@ -138,7 +138,7 @@ object Doc extends ScalaCommand[DocOptions] {
         val res = value {
           Artifacts.fetch(
             Positioned.none(Seq(dep"org.scala-lang::scaladoc:${scalaParams.scalaVersion}")),
-            build.options.finalRepositories,
+            value(build.options.finalRepositories),
             Some(scalaParams),
             logger,
             build.options.finalCache,

modules/cli/src/main/scala/scala/cli/commands/pgp/PgpExternalCommand.scala

@@ -139,7 +139,7 @@ object PgpExternalCommand {
     if (signingCliOptions.useJvm.getOrElse(false)) {
       val extraRepos =
         if (version.endsWith("SNAPSHOT"))
-          Seq(Repositories.sonatype("snapshots").root)
+          Seq(Repositories.sonatype("snapshots"))
         else
           Nil
 

modules/cli/src/main/scala/scala/cli/commands/repl/Repl.scala

@@ -371,26 +371,30 @@ object Repl extends ScalaCommand[ReplOptions] {
           value(Left(new ReplError(retCode)))
       }
 
-    def defaultArtifacts(): Either[BuildException, ReplArtifacts] =
-      ReplArtifacts.default(
-        scalaParams,
-        artifacts.userDependencies,
-        artifacts.extraClassPath,
-        logger,
-        cache,
-        options.finalRepositories,
-        addScalapy =
-          if (setupPython)
-            Some(options.notForBloopOptions.scalaPyVersion.getOrElse(Constants.scalaPyVersion))
-          else None
-      )
+    def defaultArtifacts(): Either[BuildException, ReplArtifacts] = either {
+      value {
+        ReplArtifacts.default(
+          scalaParams,
+          artifacts.userDependencies,
+          artifacts.extraClassPath,
+          logger,
+          cache,
+          value(options.finalRepositories),
+          addScalapy =
+            if (setupPython)
+              Some(options.notForBloopOptions.scalaPyVersion.getOrElse(Constants.scalaPyVersion))
+            else None
+        )
+      }
+    }
     def ammoniteArtifacts(): Either[BuildException, ReplArtifacts] =
       ReplArtifacts.ammonite(
         scalaParams,
         options.notForBloopOptions.replOptions.ammoniteVersion,
         artifacts.userDependencies,
         artifacts.extraClassPath,
         artifacts.extraSourceJars,
+        value(options.finalRepositories),
         logger,
         cache,
         directories,

modules/cli/src/main/scala/scala/cli/internal/ScalaJsLinker.scala

@@ -53,7 +53,7 @@ object ScalaJsLinker {
 
         val extraRepos =
           if (scalaJsVersion.endsWith("SNAPSHOT") || scalaJsCliVersion.endsWith("SNAPSHOT"))
-            Seq(Repositories.sonatype("snapshots").root)
+            Seq(Repositories.sonatype("snapshots"))
           else
             Nil
 

modules/cli/src/main/scala/scala/cli/launcher/LauncherCli.scala

@@ -24,7 +24,7 @@ object LauncherCli {
     val cache           = CoursierOptions().coursierCache(logger.coursierLogger(""))
     val scalaVersion    = options.cliScalaVersion.getOrElse(scalaCliScalaVersion(version))
     val scalaParameters = ScalaParameters(scalaVersion)
-    val snapshotsRepo   = Seq(Repositories.central.root, Repositories.sonatype("snapshots").root)
+    val snapshotsRepo   = Seq(Repositories.central, Repositories.sonatype("snapshots"))
 
     val cliVersion: String =
       if (version == "nightly") resolveNightlyScalaCliVersion(cache, scalaParameters) else version

modules/integration/src/test/scala/scala/cli/integration/RunTests212.scala

@@ -31,4 +31,21 @@ class RunTests212 extends RunTestDefinitions(
       )
     }
   }
+  test("2.12.nightly") {
+    TestInputs(os.rel / "sample.sc" -> "println(util.Properties.versionNumberString)").fromRoot {
+      root =>
+        val res =
+          os.proc(
+            TestUtil.cli,
+            "run",
+            ".",
+            "-S",
+            "2.12.nightly",
+            TestUtil.extraOptions
+          )
+            .call(cwd = root)
+        val version = res.out.trim()
+        expect(version.startsWith("2.12"))
+    }
+  }
 }