Skip to content

Commit 773072e

Browse files
authored
Re-organize runner folder (#5062)
This PR makes changes to `runner/` to try and make it consistent with the code layout we use for with `core/` and `libs/` * Renames * `runner` -> `runner.daemon` * `runner.client` -> `runner.launcher` * `runner.server.client` -> `runner.client` * `runner.worker-api` -> `runner.worker.api` * `runner` is now an empty module with no sources of its own * Remove the `.runner` segment in all these modules' JVM package paths
1 parent 42108d5 commit 773072e

File tree

77 files changed

+220
-210
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

77 files changed

+220
-210
lines changed

ci/mill-bootstrap.patch

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
diff --git a/mill-build/build.mill b/mill-build/build.mill
2+
index 3fc9b0097c8..5b5843a74a6 100644
3+
--- a/mill-build/build.mill
4+
+++ b/mill-build/build.mill
5+
@@ -1,5 +1,5 @@
6+
import mill._
7+
-import mill.runner.meta.MillBuildRootModule
8+
+import mill.meta.MillBuildRootModule
9+
import mill.scalalib._
10+
11+
object `package` extends MillBuildRootModule {

dist/package.mill

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ trait DistModule extends Module {
1919
def executableRaw: T[PathRef]
2020

2121
def executable = Task {
22-
Task.traverse(allPublishModules)(m => m.publishLocalCached)()
22+
Task.traverse(allPublishModules)(m => m.publishLocal(doc = false))()
2323
executableRaw()
2424
}
2525

@@ -72,7 +72,7 @@ trait DistModule extends Module {
7272

7373
object `package` extends MillJavaModule with DistModule {
7474

75-
def moduleDeps = Seq(build.runner.client)
75+
def moduleDeps = Seq(build.runner.launcher)
7676

7777
def transitiveLocalTestOverrides = Map(
7878
Task.traverse(allPublishModules)(_.localTestOverride)(): _*
@@ -133,7 +133,7 @@ object `package` extends MillJavaModule with DistModule {
133133
"-Djna.nosys=true"
134134
)
135135

136-
def mainClass = Some("mill.runner.client.MillClientMain")
136+
def mainClass = Some("mill.launcher.MillLauncherMain")
137137

138138
def executableRaw = Task {
139139
val filename = if (scala.util.Properties.isWin) "mill.bat" else "mill"
@@ -196,20 +196,20 @@ object `package` extends MillJavaModule with DistModule {
196196
prepareBootstrapLauncher(millBootstrapBat().path, Task.dest, build.millVersion(), "mill.bat")
197197
}
198198

199-
def examplePathsWithArtifactName: Task[Seq[(os.Path, String)]] = Task.Anon {
199+
def examplePathsWithArtifactName: Task[Seq[(PathRef, String)]] = Task.Input {
200200
for {
201201
exampleMod <- build.example.exampleModules
202202
path = exampleMod.moduleDir
203203
} yield {
204204
val example = path.subRelativeTo(Task.workspace)
205205
val artifactName = example.segments.mkString("-")
206-
(path, artifactName)
206+
(PathRef(path), artifactName)
207207
}
208208
}
209209

210210
def exampleZips: T[Seq[PathRef]] = Task {
211211
examplePathsWithArtifactName().map { case (examplePath, exampleStr) =>
212-
os.copy(examplePath, Task.dest / exampleStr, createFolders = true)
212+
os.copy(examplePath.path, Task.dest / exampleStr, createFolders = true)
213213
val ignoreErrorsOnCI = Task.dest / exampleStr / "ignoreErrorsOnCI"
214214
if (os.exists(ignoreErrorsOnCI)) os.remove(ignoreErrorsOnCI)
215215
os.write(Task.dest / exampleStr / ".mill-version", build.millLastTag())
@@ -255,9 +255,9 @@ object `package` extends MillJavaModule with DistModule {
255255
// Use assembly jar as the upstream ivy classpath rather than using runClasspath
256256
// directly to try and avoid native image command length problems on windows
257257
def nativeImageClasspath =
258-
Seq(build.runner.client.resolvedIvyAssembly().pathRef) ++
259-
build.runner.client.upstreamLocalAssemblyClasspath() ++
260-
build.runner.client.localClasspath()
258+
Seq(build.runner.launcher.resolvedIvyAssembly().pathRef) ++
259+
build.runner.launcher.upstreamLocalAssemblyClasspath() ++
260+
build.runner.launcher.localClasspath()
261261

262262
def localBinName = "mill-native"
263263

example/extending/metabuild/4-meta-build/mill-build/build.mill

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
package build
22
import mill._, scalalib._
3-
import mill.runner.meta.MillBuildRootModule
3+
import mill.meta.MillBuildRootModule
44

55
object `package` extends MillBuildRootModule {
66
val scalatagsVersion = "0.13.1"
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
package build
22
import mill._
3-
import mill.runner.meta.MillBuildRootModule
3+
import mill.meta.MillBuildRootModule
44

55
object `package` extends MillBuildRootModule

example/extending/plugins/7-writing-mill-plugins/build.mill

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,8 +31,8 @@ object myplugin extends ScalaModule with PublishModule {
3131

3232
// Create a Mill executable configured for testing our plugin
3333
object millExecutable extends JavaModule {
34-
def mvnDeps = Seq(mvn"com.lihaoyi:mill-runner-client_3:$millVersion")
35-
def mainClass = Some("mill.runner.client.MillClientMain")
34+
def mvnDeps = Seq(mvn"com.lihaoyi:mill-runner-launcher_3:$millVersion")
35+
def mainClass = Some("mill.launcher.MillLauncherMain")
3636
}
3737
}
3838

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,10 @@
11
package build
22

33
import mill.*
4+
import mill.meta.MillBuildRootModule
45
import mill.scalalib.*
56

6-
object `package` extends mill.runner.meta.MillBuildRootModule:
7+
object `package` extends MillBuildRootModule:
78
def mvnDeps = Seq(
89
mvn"org.scala-lang::toolkit:0.5.0"
910
)
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
package build
22
import mill.*
3+
import mill.meta.MillBuildRootModule
34
import mill.scalalib.*
4-
import mill.runner.meta.MillBuildRootModule
55

66
object `package` extends MillBuildRootModule

integration/ide/gen-idea/resources/extended/idea/mill_modules/mill-build.mill-build.iml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -79,8 +79,8 @@
7979
<orderEntry type="library" name="mill-libs_3.jar" level="project"/>
8080
<orderEntry type="library" name="mill-moduledefs_3-0.11.3-M5.jar" level="project"/>
8181
<orderEntry type="library" name="mill-runner-codesig_3.jar" level="project"/>
82+
<orderEntry type="library" name="mill-runner-compilerworker-api_3.jar" level="project"/>
8283
<orderEntry type="library" name="mill-runner-meta_3.jar" level="project"/>
83-
<orderEntry type="library" name="mill-runner-worker-api_3.jar" level="project"/>
8484
<orderEntry type="library" name="native-terminal-no-ffm-0.0.9.1.jar" level="project"/>
8585
<orderEntry type="library" name="os-lib_3-0.11.5-M2.jar" level="project"/>
8686
<orderEntry type="library" name="plexus-archiver-4.10.0.jar" level="project"/>

integration/ide/gen-idea/resources/extended/mill-build/build.mill

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import mill._
2+
import mill.meta.MillBuildRootModule
23
import mill.scalalib._
3-
import mill.runner.meta.MillBuildRootModule
44

55
object `package` extends MillBuildRootModule {
66
def mvnDeps = Seq(mvn"org.scalameta::munit:0.7.29")

integration/invalidation/multi-level-editing/resources/mill-build/build.mill

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
package build
22

33
import mill._, scalalib._
4-
import mill.runner.meta.MillBuildRootModule
4+
import mill.meta.MillBuildRootModule
55

66
object `package` extends MillBuildRootModule {
77
def mvnDeps = Seq(mvn"com.lihaoyi::scalatags:${constant.MetaConstant.scalatagsVersion}")

0 commit comments

Comments
 (0)