Revised build importer implementations

.scalafmt.conf

@@ -29,6 +29,9 @@ project {
     "glob:**/example/**/*-spotless*/**",
     "glob:**/javalib/test/resources/checkstyle/**",
     "glob:**/init/test/resources/giter8/hello.g8/src/main/g8/build.mill",
+    "glob:**/init/gradle/test/resources/**",
+    "glob:**/init/maven/test/resources/**",
+    "glob:**/init/sbt/test/resources/**",
     "glob:**/javalib/test/resources/javalib/palantirformat/**",
     "glob:**/integration/failure/parse-error/**",
     "glob:**/testkit/test/resources/example-test-example-project/build.mill",

example/migrating/javalib/3-maven-complete-large/build.mill

@@ -6,7 +6,9 @@
 > git remote add -f origin https://github.com/iluwatar/java-design-patterns
 > git checkout ede37bd05568b1b8b814d8e9a1d2bbd71d9d615d
 
-> ./mill init --jvm-id 21 # Repo needs Java >=21 to build and test
+> ./mill init
+
+> echo 21 > .mill-jvm-version # Repo needs Java >=21 to build and test
 
 > rm twin/src/test/java/com/iluwatar/twin/BallThreadTest.java # skip flaky test
 > rm actor-model/src/test/java/com/iluwatar/actor/ActorModelTest.java # skip flaky test

example/migrating/javalib/4-gradle-complete/build.mill

@@ -6,7 +6,7 @@
 > git remote add -f origin https://github.com/komamitsu/fluency.git
 > git checkout 2.7.3 # multi-module Java project that requires Java 16+
 
-> ./mill init --base-module FluencyModule --jvm-id 16
+> ./mill init
 converting Gradle build
 ...
 init completed, run "mill resolve _" to list available tasks

example/migrating/javalib/5-gradle-incomplete/build.mill

@@ -4,13 +4,12 @@
 
 > git init .
 > git remote add -f origin https://github.com/mockito/mockito.git
-> git checkout v5.15.2 # multi-module Java project that requires Java 17+
+> git checkout v5.19.0 # multi-module Java project
 
-> ./mill init --base-module MockitoModule --jvm-id 17   # init ignores custom dependency configurations
+> ./mill init --gradle-jvm-id 17  # imported modules are not fully functional
 converting Gradle build
-ignoring errorprone dependency (com.google.errorprone,error_prone_core,2.23.0)
 init completed, run "mill resolve _" to list available tasks
 
-> ./mill __.compile   # compilation error can be by fixed by using the Mill plugin for ErrorProne
-error: plug-in not found: ErrorProne
+> ./mill mockito-core.compile   # module jvmId needs tweaking
+error: java.lang.UnsupportedClassVersionError: com/google/errorprone/ErrorProneJavacPlugin has been compiled by a more recent version...
 */

example/migrating/scalalib/2-sbt-incomplete/build.mill

@@ -3,17 +3,15 @@
 > rm build.mill # remove any existing build file
 
 > git init .
-> git remote add -f origin https://github.com/tototoshi/scala-csv.git
-> git checkout 2.0.0
+> git remote add -f origin https://github.com/fthomas/refined.git
+> git checkout v0.11.3
 
 > ./mill init
 converting sbt build
 ...
 init completed, run "mill resolve _" to list available tasks
 
-> ./mill compile # You will have to further configure the `CrossScalaModule` for different Scala versions
-compiling 7 Scala sources and 3 Java sources to ...
-error: class CSVReader protected (private val lineReader: LineReader)(implicit format: CSVFormat) extends Closeable with CSVReaderCompat {
-error:                                                                                                                   ^
-error: one error found
+> ./mill modules.core.jvm[2.12.20].compile # source roots for custom version ranges need tweaking
+compiling 22 Scala sources to ...
+error: ...object boolean is not a member of package eu.timepit.refined
 */

integration/manual/migrating/src/MillInitGradleAsmTests.scala

@@ -0,0 +1,13 @@
+package mill.integration
+import utest.*
+object MillInitGradleAsmTests extends MillInitTestSuite {
+  def tests = Tests {
+    test - checkImport(
+      gitUrl = "https://gitlab.ow2.org/asm/asm.git",
+      gitBranch = "ASM_9_8",
+      initArgs = Seq("--gradle-jvm-id", "11"),
+      passingTasks = Seq("asm.compile"),
+      failingTasks = Seq("tools.retrofitter.compile")
+    )
+  }
+}

integration/manual/migrating/src/MillInitGradleEhcache3Tests.scala

@@ -0,0 +1,12 @@
+package mill.integration
+import utest.*
+object MillInitGradleEhcache3Tests extends MillInitTestSuite {
+  def tests = Tests {
+    test - checkImport(
+      gitUrl = "https://github.com/ehcache/ehcache3.git",
+      gitBranch = "v3.10.8",
+      initArgs = Seq("--gradle-jvm-id", "11"),
+      failingTasks = Seq("ehcache-api.compile")
+    )
+  }
+}

integration/manual/migrating/src/MillInitGradleFastCsvTests.scala

@@ -0,0 +1,12 @@
+package mill.integration
+import utest.*
+object MillInitGradleFastCsvTests extends MillInitTestSuite {
+  def tests = Tests {
+    test - checkImport(
+      gitUrl = "https://github.com/osiegmar/FastCSV.git",
+      gitBranch = "v4.0.0",
+      initArgs = Seq("--gradle-jvm-id", "24"),
+      failingTasks = Seq("lib.compile")
+    )
+  }
+}

integration/manual/migrating/src/MillInitGradleJCommanderTests.scala

@@ -0,0 +1,12 @@
+package mill.integration
+import utest.*
+object MillInitGradleJCommanderTests extends MillInitTestSuite {
+  def tests = Tests {
+    test - checkImport(
+      gitUrl = "https://github.com/cbeust/jcommander.git",
+      gitBranch = "2.0",
+      initArgs = Seq("--gradle-jvm-id", "11"),
+      failingTasks = Seq("test.compile")
+    )
+  }
+}

integration/manual/migrating/src/MillInitGradleMicroconfigTests.scala

@@ -0,0 +1,12 @@
+package mill.integration
+import utest.*
+object MillInitGradleMicroconfigTests extends MillInitTestSuite {
+  def tests = Tests {
+    test - checkImport(
+      gitUrl = "https://github.com/microconfig/microconfig.git",
+      gitBranch = "v4.9.5",
+      initArgs = Seq("--merge"),
+      passingTasks = Seq("_.test")
+    )
+  }
+}

integration/manual/migrating/src/MillInitGradleMockitoTests.scala

@@ -0,0 +1,12 @@
+package mill.integration
+import utest.*
+object MillInitGradleMockitoTests extends MillInitTestSuite {
+  def tests = Tests {
+    test - checkImport(
+      gitUrl = "https://github.com/mockito/mockito.git",
+      gitBranch = "v5.19.0",
+      initArgs = Seq("--gradle-jvm-id", "17"),
+      failingTasks = Seq("mockito-core.compile")
+    )
+  }
+}

integration/manual/migrating/src/MillInitGradleSpotbugsTests.scala

@@ -0,0 +1,13 @@
+package mill.integration
+import utest.*
+object MillInitGradleSpotbugsTests extends MillInitTestSuite {
+  def tests = Tests {
+    test - checkImport(
+      gitUrl = "https://github.com/spotbugs/spotbugs.git",
+      gitBranch = "4.9.4",
+      initArgs = Seq("--gradle-jvm-id", "17"),
+      passingTasks = Seq("spotbugs.compile"),
+      failingTasks = Seq("eclipsePlugin-test.resolvedMvnDeps")
+    )
+  }
+}

integration/manual/migrating/src/MillInitGradleSpringFrameworkTests.scala

@@ -0,0 +1,12 @@
+package mill.integration
+import utest.*
+object MillInitGradleSpringFrameworkTests extends MillInitTestSuite {
+  def tests = Tests {
+    test - checkImport(
+      gitUrl = "https://github.com/spring-projects/spring-framework.git",
+      gitBranch = "v6.2.11",
+      initArgs = Seq("--gradle-jvm-id", "24"),
+      failingTasks = Seq("spring-instrument.resolvedMvnDeps")
+    )
+  }
+}

integration/manual/migrating/src/MillInitMavenByteBuddyTests.scala

@@ -0,0 +1,12 @@
+package mill.integration
+import utest.*
+object MillInitMavenByteBuddyTests extends MillInitTestSuite {
+  def tests = Tests {
+    test - checkImport(
+      gitUrl = "https://github.com/raphw/byte-buddy.git",
+      gitBranch = "byte-buddy-1.17.7",
+      passingTasks = Seq("byte-buddy.compile"),
+      failingTasks = Seq("byte-buddy-android-test.compile")
+    )
+  }
+}

integration/manual/migrating/src/MillInitMavenCheckstyleTests.scala

@@ -0,0 +1,11 @@
+package mill.integration
+import utest.*
+object MillInitMavenCheckstyleTests extends MillInitTestSuite {
+  def tests = Tests {
+    test - checkImport(
+      "https://github.com/checkstyle/checkstyle.git",
+      "checkstyle-11.0.0",
+      failingTasks = Seq("compile")
+    )
+  }
+}

integration/manual/migrating/src/MillInitMavenErrorProneTests.scala

@@ -0,0 +1,11 @@
+package mill.integration
+import utest.*
+object MillInitMavenErrorProneTests extends MillInitTestSuite {
+  def tests = Tests {
+    test - checkImport(
+      "https://github.com/google/error-prone.git",
+      "v2.41.0",
+      failingTasks = Seq("annotation.javaHome")
+    )
+  }
+}

integration/manual/migrating/src/MillInitMavenJansiTests.scala

@@ -0,0 +1,11 @@
+package mill.integration
+import utest.*
+object MillInitMavenJansiTests extends MillInitTestSuite {
+  def tests = Tests {
+    test - checkImport(
+      "https://github.com/fusesource/jansi.git",
+      "jansi-2.4.2",
+      passingTasks = Seq("test")
+    )
+  }
+}

integration/manual/migrating/src/MillInitMavenJodaBeansTests.scala

@@ -0,0 +1,11 @@
+package mill.integration
+import utest.*
+object MillInitMavenJodaBeansTests extends MillInitTestSuite {
+  def tests = Tests {
+    test - checkImport(
+      "https://github.com/JodaOrg/joda-beans.git",
+      "v2.11.1",
+      passingTasks = Seq("test")
+    )
+  }
+}

integration/manual/migrating/src/MillInitMavenNettyTests.scala

@@ -0,0 +1,11 @@
+package mill.integration
+import utest.*
+object MillInitMavenNettyTests extends MillInitTestSuite {
+  def tests = Tests {
+    test - checkImport(
+      "https://github.com/netty/netty.git",
+      "netty-4.2.6.Final",
+      failingTasks = Seq("buffer.test.discoveredTestClasses")
+    )
+  }
+}

integration/manual/migrating/src/MillInitMavenSpringAiTests.scala

@@ -0,0 +1,11 @@
+package mill.integration
+import utest.*
+object MillInitMavenSpringAiTests extends MillInitTestSuite {
+  def tests = Tests {
+    test - checkImport(
+      "https://github.com/spring-projects/spring-ai.git",
+      "v1.0.3",
+      failingTasks = Seq("spring-ai-commons.resolvedMvnDeps")
+    )
+  }
+}

integration/manual/migrating/src/MillInitSbtAirstreamTests.scala

@@ -0,0 +1,13 @@
+package mill.integration
+import utest.*
+object MillInitSbtAirstreamTests extends MillInitTestSuite {
+  def tests = Tests {
+    test - checkImport(
+      "https://github.com/raquo/Airstream.git",
+      "v17.2.1",
+      failingTasks = Seq(
+        ("[2.13.16].test.testOnly", "com.raquo.airstream.web.WebStorageVarSpec")
+      )
+    )
+  }
+}

integration/manual/migrating/src/MillInitSbtCatsTests.scala

@@ -0,0 +1,11 @@
+package mill.integration
+import utest.*
+object MillInitSbtCatsTests extends MillInitTestSuite {
+  def tests = Tests {
+    test - checkImport(
+      "https://github.com/typelevel/cats.git",
+      "v2.13.0",
+      failingTasks = Seq("kernel.jvm[2.13.16].compile")
+    )
+  }
+}

integration/manual/migrating/src/MillInitSbtEnumeratumTests.scala

@@ -0,0 +1,12 @@
+package mill.integration
+import utest.*
+object MillInitSbtEnumeratumTests extends MillInitTestSuite {
+  def tests = Tests {
+    test - checkImport(
+      "https://github.com/lloydmeta/enumeratum.git",
+      "enumeratum-1.9.0",
+      passingTasks = Seq("enumeratum-core.jvm[2.13.16].compile"),
+      failingTasks = Seq("macros.jvm[2.13.16].compile")
+    )
+  }
+}

integration/manual/migrating/src/MillInitSbtFs2Tests.scala

@@ -0,0 +1,18 @@
+package mill.integration
+import utest.*
+object MillInitSbtFs2Tests extends MillInitTestSuite {
+  def tests = Tests {
+    test - checkImport(
+      "https://github.com/typelevel/fs2.git",
+      "v3.12.0",
+      passingTasks = Seq(
+        ("core.js[2.13.16].test.testOnly", "fs2.hashing.HashingSuite"),
+        ("core.jvm[3.3.5].test.testOnly", "fs2.hashing.HashingSuite")
+      ),
+      failingTasks = Seq(
+        "core.native[2.12.20].test.scalaNativeWorkerClasspath",
+        "benchmark[3.3.5].compile"
+      )
+    )
+  }
+}

integration/manual/migrating/src/MillInitSbtGatlingTests.scala

@@ -0,0 +1,11 @@
+package mill.integration
+import utest.*
+object MillInitSbtGatlingTests extends MillInitTestSuite {
+  def tests = Tests {
+    test - checkImport(
+      "https://github.com/gatling/gatling.git",
+      "v3.14.3",
+      failingTasks = Seq("gatling-http-client.test.compile")
+    )
+  }
+}

integration/manual/migrating/src/MillInitSbtLilaTests.scala

@@ -0,0 +1,11 @@
+package mill.integration
+import utest.*
+object MillInitSbtLilaTests extends MillInitTestSuite {
+  def tests = Tests {
+    test - checkImport(
+      "https://github.com/lichess-org/lila.git",
+      "master",
+      failingTasks = Seq(("resolve", "_"))
+    )
+  }
+}

integration/manual/migrating/src/MillInitSbtNscalaTimeTests.scala

@@ -0,0 +1,11 @@
+package mill.integration
+import utest.*
+object MillInitSbtNscalaTimeTests extends MillInitTestSuite {
+  def tests = Tests {
+    test - checkImport(
+      "https://github.com/nscala-time/nscala-time.git",
+      "releases/3.0.0",
+      passingTasks = Seq("_.test")
+    )
+  }
+}

integration/manual/migrating/src/MillInitSbtRefinedTests.scala

@@ -0,0 +1,12 @@
+package mill.integration
+import utest.*
+object MillInitSbtRefinedTests extends MillInitTestSuite {
+  def tests = Tests {
+    test - checkImport(
+      "https://github.com/fthomas/refined.git",
+      "v0.11.3",
+      passingTasks = Seq("__.showModuleDeps"),
+      failingTasks = Seq("modules.core.jvm[3.3.4].compile")
+    )
+  }
+}

integration/manual/migrating/src/MillInitSbtScala3Tests.scala

@@ -0,0 +1,14 @@
+package mill.integration
+
+import utest.*
+
+object MillInitSbtScala3Tests extends MillInitTestSuite {
+  def tests = Tests {
+    // sbt 1.11.0
+    test - checkImport(
+      "https://github.com/scala/scala3.git",
+      "3.7.1",
+      failingTasks = Seq(("resolve", "_"))
+    )
+  }
+}

integration/manual/migrating/src/MillInitSbtScalaLoggingTests.scala

@@ -0,0 +1,13 @@
+package mill.integration
+
+import utest.*
+
+object MillInitSbtScalaLoggingTests extends MillInitTestSuite {
+  def tests = Tests {
+    test - checkImport(
+      "https://github.com/lightbend-labs/scala-logging.git",
+      "v3.9.5",
+      passingTasks = Seq("_.test")
+    )
+  }
+}