Skip to content

Commit be62277

Browse files
committed
drop compilation for dotty (<= 0.27) (Scala 3 compilation remains)
1 parent fd7dd86 commit be62277

File tree

11 files changed

+12
-45
lines changed

11 files changed

+12
-45
lines changed

build.sbt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -500,7 +500,6 @@ lazy val runtimeDependencies =
500500
Dependencies.sbtLaunch -> Some("launcher/sbt-launch.jar"),
501501
Dependencies.sbtInterface -> Some("lib/jps/sbt-interface.jar"),
502502
Dependencies.zincInterface -> Some("lib/jps/compiler-interface.jar"),
503-
Dependencies.dottySbtBridge -> Some("lib/jps/dotty-sbt-bridge.jar"),
504503
Dependencies.scala3SbtBridge -> Some("lib/jps/scala3-sbt-bridge.jar"),
505504
Dependencies.compilerBridgeSources_2_13 -> Some("lib/jps/compiler-interface-sources-2.13.jar"),
506505
Dependencies.compilerBridgeSources_2_11 -> Some("lib/jps/compiler-interface-sources-2.11.jar"),

project/dependencies.scala

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,6 @@ object Dependencies {
112112
val compilerBridgeSources_2_10 = "org.scala-sbt" % "compiler-bridge_2.10" % zincVersion classifier "sources"
113113
val compilerBridgeSources_2_11 = "org.scala-sbt" % "compiler-bridge_2.11" % zincVersion classifier "sources"
114114
val compilerBridgeSources_2_13 = "org.scala-sbt" % "compiler-bridge_2.13" % zincVersion classifier "sources"
115-
val dottySbtBridge = "ch.epfl.lamp" % "dotty-sbt-bridge" % "0.27.0-RC1"
116115
val scala3SbtBridge = "org.scala-lang" % "scala3-sbt-bridge" % "3.0.0-M2"
117116

118117
// "provided" danger: we statically depend on a single version, but need to support all the version
@@ -198,7 +197,6 @@ object DependencyGroups {
198197
compilerBridgeSources_2_10,
199198
compilerBridgeSources_2_11,
200199
compilerBridgeSources_2_13,
201-
dottySbtBridge,
202200
scala3SbtBridge
203201
)
204202
}

scala/compiler-jps/src/org/jetbrains/jps/incremental/scala/IdeaIncrementalBuilder.scala

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,8 +73,7 @@ class IdeaIncrementalBuilder(category: BuilderCategory) extends ModuleLevelBuild
7373

7474
val successfullyCompiled = mutable.Set.empty[File]
7575

76-
val compilerName = if (data.CompilerDataFactory.hasDotty(modules)) "dotc"
77-
else "scalac"
76+
val compilerName = "scalac"
7877

7978
val client = new local.IdeClientIdea(compilerName, context, chunk, outputConsumer,
8079
callback, successfullyCompiled, packageObjectsData)

scala/compiler-jps/src/org/jetbrains/jps/incremental/scala/data/ArgumentsParser.scala

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,6 @@ object ArgumentsParser
2727
PathToFile(scalaBridgeSourceJar_2_10),
2828
PathToFile(scalaBridgeSourceJar_2_11),
2929
PathToFile(scalaBridgeSourceJar_2_13),
30-
PathToFile(dottyBridgeJar),
3130
PathToFile(scala3BridgeJar),
3231
PathToFile(interfacesHome),
3332
javaClassVersion,
@@ -59,7 +58,6 @@ object ArgumentsParser
5958
_2_13 = scalaBridgeSourceJar_2_13
6059
)
6160
val dottyBridges = SbtData.Scala3Jars(
62-
dotty = dottyBridgeJar,
6361
scala3 = scala3BridgeJar
6462
)
6563
val compilerBridges = SbtData.CompilerBridges(scalaBridgeSources, dottyBridges)

scala/compiler-jps/src/org/jetbrains/jps/incremental/scala/data/CompilerDataFactory.scala

Lines changed: 3 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -96,17 +96,9 @@ object CompilerDataFactory
9696
}
9797
}
9898

99-
def hasDotty(modules: Set[JpsModule]): Boolean =
100-
modules.exists {
101-
compilerJarsIn(_).exists(_.hasDotty)
102-
}
10399
def hasScala3(modules: Set[JpsModule]): Boolean =
104100
modules.exists {
105-
compilerJarsIn(_).exists(_.hasDotty)
106-
}
107-
def hasScala3OrDotty(modules: Set[JpsModule]): Boolean =
108-
modules.exists {
109-
compilerJarsIn(_).exists(_.hasScala3OrDotty)
101+
compilerJarsIn(_).exists(_.hasScala3)
110102
}
111103

112104
private def hasOldScala(modules: Set[JpsModule]) =
@@ -123,11 +115,11 @@ object CompilerDataFactory
123115
def scalaOptionsFor(compilerSettings: CompilerSettings, chunk: ModuleChunk): Seq[String] = {
124116
val configuredOptions = compilerSettings.getCompilerOptions
125117
val modules = chunk.getModules.asScala.toSet
126-
val hasScala3OrDotty = CompilerDataFactory.hasScala3OrDotty(modules)
118+
val hasScala3 = CompilerDataFactory.hasScala3(modules)
127119

128120
bootClasspathOptions(hasOldScala(modules)) ++
129121
semanticDbOptionsFor(configuredOptions.toIndexedSeq, chunk) ++
130-
configuredOptions.filterNot(_.startsWith("-g:") && hasScala3OrDotty) // TODO SCL-16881
122+
configuredOptions.filterNot(_.startsWith("-g:") && hasScala3) // TODO SCL-16881
131123
}
132124

133125
private def bootClasspathOptions(hasOldScala: Boolean): Seq[String] =

scala/compiler-jps/src/org/jetbrains/jps/incremental/scala/local/CompilerFactoryImpl.scala

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,6 @@ class CompilerFactoryImpl(sbtData: SbtData) extends CompilerFactory {
5252
scalaInstance = scalaInstance,
5353
javaClassVersion = sbtData.javaClassVersion,
5454
client = Option(client),
55-
isDotty = compilerJars.exists(_.hasDotty),
5655
isScala3 = compilerJars.exists(_.hasScala3),
5756
)
5857

@@ -114,11 +113,8 @@ object CompilerFactoryImpl {
114113
scalaInstance: ScalaInstance,
115114
javaClassVersion: String,
116115
client: Option[Client],
117-
isDotty: Boolean,
118116
isScala3: Boolean): File =
119-
if (isDotty)
120-
compilerBridges.scala3.dotty
121-
else if (isScala3)
117+
if (isScala3)
122118
compilerBridges.scala3.scala3
123119
else {
124120
val scalaVersion = scalaInstance.actualVersion

scala/compiler-shared/src/org/jetbrains/jps/incremental/scala/package.scala

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -25,15 +25,8 @@ package object scala {
2525
def |>[R](f: T => R): R = f(v)
2626
}
2727

28-
def containsDotty(files: Iterable[File]): Boolean =
29-
files.exists(_.getName.startsWith("dotty"))
3028
def containsScala3(files: Iterable[File]): Boolean =
3129
files.exists(_.getName.startsWith("scala3"))
32-
def containsScala3OrDotty(files: Iterable[File]): Boolean =
33-
files.exists(f => {
34-
val name = f.getName
35-
name.startsWith("scala3") || name.startsWith("dotty")
36-
})
3730

3831
// TODO implement a better version comparison
3932
def compilerVersionIn(compiler: File, versions: String*): Boolean =

scala/compiler-shared/src/org/jetbrains/plugins/scala/compiler/data/Arguments.scala

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,6 @@ case class Arguments(sbtData: SbtData,
2929
fileToPath(sbtData.compilerBridges.scala._2_10),
3030
fileToPath(sbtData.compilerBridges.scala._2_11),
3131
fileToPath(sbtData.compilerBridges.scala._2_13),
32-
fileToPath(sbtData.compilerBridges.scala3.dotty),
3332
fileToPath(sbtData.compilerBridges.scala3.scala3),
3433
fileToPath(sbtData.interfacesHome),
3534
sbtData.javaClassVersion,

scala/compiler-shared/src/org/jetbrains/plugins/scala/compiler/data/CompilerJars.scala

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
package org.jetbrains.plugins.scala.compiler.data
22

3-
import java.io.File
3+
import org.jetbrains.jps.incremental.scala.containsScala3
44

5-
import org.jetbrains.jps.incremental.scala.{containsDotty, containsScala3, containsScala3OrDotty}
5+
import java.io.File
66

77
/**
88
* @author Pavel Fatin
@@ -11,12 +11,8 @@ case class CompilerJars(libraries: Seq[File],
1111
compiler: File,
1212
extra: Seq[File]) {
1313

14-
def hasDotty: Boolean =
15-
containsDotty(extra)
1614
def hasScala3: Boolean =
1715
containsScala3(extra)
18-
def hasScala3OrDotty: Boolean =
19-
containsScala3OrDotty(extra)
2016

2117
def allJars: Seq[File] =
2218
libraries ++ extra :+ compiler

scala/compiler-shared/src/org/jetbrains/plugins/scala/compiler/data/CompilerJarsFactory.scala

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
package org.jetbrains.plugins.scala.compiler.data
22

3-
import java.io.File
4-
5-
import org.jetbrains.jps.incremental.scala.{compilerVersionIn, containsDotty, containsScala3}
3+
import org.jetbrains.jps.incremental.scala.{compilerVersionIn, containsScala3}
64
import org.jetbrains.plugins.scala.compiler.data.CompilerJarsFactory.CompilerJarsResolveError
75
import org.jetbrains.plugins.scala.util.JarUtil
86
import org.jetbrains.plugins.scala.util.JarUtil.JarFileWithName
97

8+
import java.io.File
9+
1010
trait CompilerJarsFactory {
1111

1212
def fromFiles(files: Seq[File]): Either[CompilerJarsResolveError, CompilerJars]
@@ -31,8 +31,7 @@ object CompilerJarsFactory
3131
def fromJarFiles(files: Seq[JarFileWithName]): Either[CompilerJarsResolveError, CompilerJars] = {
3232
val ioFiles = files.map(_.file)
3333
val compilerPrefix =
34-
if (containsDotty(ioFiles)) "dotty"
35-
else if (containsScala3(ioFiles)) "scala3"
34+
if (containsScala3(ioFiles)) "scala3"
3635
else "scala"
3736

3837
val init: Either[CompilerJarsResolveError, Seq[JarFileWithName]] = Right(Seq.empty)

scala/compiler-shared/src/org/jetbrains/plugins/scala/compiler/data/SbtData.scala

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ object SbtData {
3232
/**
3333
* Contains already compiled dotty/scala3 compiler bridges.
3434
*/
35-
case class Scala3Jars(dotty: File, scala3: File)
35+
case class Scala3Jars(scala3: File)
3636

3737
val compilerInterfacesKey = "scala.compiler.interfaces.dir"
3838

@@ -61,7 +61,6 @@ object SbtData {
6161
scalaBridge_2_10 <- fileWithName("compiler-interface-sources-2.10.jar")
6262
scalaBridge_2_11 <- fileWithName("compiler-interface-sources-2.11.jar")
6363
scalaBridge_2_13 <- fileWithName("compiler-interface-sources-2.13.jar")
64-
dottyBridge <- fileWithName("dotty-sbt-bridge.jar")
6564
scala3Bridge <- fileWithName("scala3-sbt-bridge.jar")
6665
sbtVersion <- readSbtVersionFrom(sbtInterfaceJar)
6766
} yield {
@@ -73,7 +72,6 @@ object SbtData {
7372
_2_13 = scalaBridge_2_13
7473
)
7574
val dottyBridges = Scala3Jars(
76-
dotty = dottyBridge,
7775
scala3 = scala3Bridge
7876
)
7977
val compilerBridges = CompilerBridges(scalaBridgeSources, dottyBridges)

0 commit comments

Comments
 (0)