Inline repl project into dotty-compiler

As discussed with @smarter, this has the effect that if you depend on
dotty-compiler version X, you also get the appropriate dotty repl with
the same version

Author: felixmulder

build.sbt

@@ -18,7 +18,6 @@ val `dotty-bench-bootstrapped` = Build.`dotty-bench-bootstrapped`
 val `scala-library` = Build.`scala-library`
 val `scala-compiler` = Build.`scala-compiler`
 val `scala-reflect` = Build.`scala-reflect`
-val `dotty-repl` = Build.`dotty-repl`
 val scalap = Build.scalap
 val dist = Build.dist
 val `dist-bootstrapped` = Build.`dist-bootstrapped`

repl/src/dotty/tools/repl/AbstractFileClassLoader.scala renamed to compiler/src/dotty/tools/repl/AbstractFileClassLoader.scala

@@ -17,11 +17,11 @@ import dotc.core.Symbols.{ Symbol, ClassSymbol, defn }
 import dotc.core.SymDenotations.NoDenotation
 import dotc.core.Types._
 import dotc.printing.Texts._
-import dotc.printing.{ GlobalPrec, DotPrec, Printer, PlainPrinter }
+import dotc.printing.{ GlobalPrec, DotPrec, Printer, PlainPrinter, RefinedPrinter }
 import dotc.typer.Implicits.SearchResult
 import dotc.typer.ImportInfo
 
-class UserFacingPrinter(_ctx: Context) extends PlainPrinter(_ctx) {
+class UserFacingPrinter(_ctx: Context) extends RefinedPrinter(_ctx) {
 
   private def panic(msg: String): Nothing = throw new AssertionError(msg)
 
@@ -75,15 +75,6 @@ class UserFacingPrinter(_ctx: Context) extends PlainPrinter(_ctx) {
       }
     }
 
-  override def toText(denot: Denotation): Text = denot match {
-    case NoDenotation =>
-      panic("NoDenotation encountered in UserFacingPrinter")
-    case denot: MultiDenotation =>
-      panic("MultiDenotation not allowed in UserFacingPrinter")
-    case _ =>
-      toText(denot.symbol)
-  }
-
   override def toText(const: Constant): Text = Str(const.value.toString)
 
   override def toText(tp: Type): Text = tp match {
@@ -142,7 +133,6 @@ class UserFacingPrinter(_ctx: Context) extends PlainPrinter(_ctx) {
         printPkg(tp.cls.owner.asClass) ~ "." ~ nameString(tp.cls.name)
       }
     }
+    case tp => super.toText(tp)
   }
-
-  override lazy val plain = new PlainPrinter(_ctx)
 }

repl/src/dotty/tools/repl/AmmoniteReader.scala renamed to compiler/src/dotty/tools/repl/AmmoniteReader.scala

@@ -130,6 +130,7 @@ object Build {
     javaSource        in Compile    := baseDirectory.value / "src",
     javaSource        in Test       := baseDirectory.value / "test",
     resourceDirectory in Compile    := baseDirectory.value / "resources",
+    resourceDirectory in Test       := baseDirectory.value / "test-resources",
 
     // Prevent sbt from rewriting our dependencies
     ivyScala ~= (_ map (_ copy (overrideScalaVersion = false)))
@@ -358,47 +359,6 @@ object Build {
     )
   )
 
-  lazy val `dotty-repl` = project.in(file("repl")).
-    dependsOn(
-      `dotty-compiler`,
-      `dotty-compiler` % "test->test"
-    ).
-    settings(commonNonBootstrappedSettings).
-    settings(
-      // set system in/out for repl
-      connectInput in run := true,
-      outputStrategy := Some(StdoutOutput),
-
-      // enable verbose exception messages for JUnit
-      testOptions in Test += Tests.Argument(TestFrameworks.JUnit, "-a", "-v"),
-
-      resourceDirectory in Test := baseDirectory.value / "test-resources",
-
-      run := Def.inputTaskDyn {
-        val classPath =
-          (packageAll in `dotty-compiler-bootstrapped`).value("dotty-library") + ":" +
-          (packageAll in `dotty-compiler-bootstrapped`).value("dotty-interfaces")
-
-        val args: Seq[String] = spaceDelimited("<arg>").parsed
-
-        val fullArgs = args.span(_ != "-classpath") match {
-          case (beforeCp, Nil) => beforeCp ++ ("-classpath" :: classPath :: Nil)
-          case (beforeCp, rest) => beforeCp ++ rest
-        }
-
-        (runMain in Compile).toTask(
-          s" dotty.tools.repl.Main " + fullArgs.mkString(" ")
-        )
-      }.evaluated,
-
-      repl := run.evaluated,
-
-      javaOptions ++= (javaOptions in `dotty-compiler`).value,
-      fork in run := true,
-      fork in Test := true,
-      parallelExecution in Test := false
-    )
-
   lazy val `dotty-doc` = project.in(file("doc-tool")).
     dependsOn(`dotty-compiler`, `dotty-compiler` % "test->test").
     settings(commonNonBootstrappedSettings).
@@ -409,7 +369,6 @@ object Build {
     settings(commonBootstrappedSettings).
     settings(dottyDocSettings)
 
-
   lazy val `dotty-bot` = project.in(file("bot")).
     settings(commonScala2Settings).
     settings(