Forward port sbt/sbt#3701, JDK9/Scala 2.{10,11} overcompilation

sbt · Nov 3, 2017 · 641a491 · 641a491
1 parent 8c42c1f
commit 641a491
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 4 deletions.
diff --git a/internal/compiler-bridge/src/main/scala/xsbt/Dependency.scala b/internal/compiler-bridge/src/main/scala/xsbt/Dependency.scala
@@ -114,8 +114,9 @@ final class Dependency(val global: CallbackGlobal) extends LocateClassFile with
             // The dependency comes from a JAR
             for {
               zip <- zipEntry.underlyingSource
-              classFile <- Option(zip.file)
-            } binaryDependency(classFile, binaryClassName)
+              jarFile <- Option(zip.file)
+              if !jarFile.isDirectory // workaround for JDK9 and Scala 2.10/2.11, see https://github.com/sbt/sbt/pull/3701
+            } binaryDependency(jarFile, binaryClassName)
           case pf: PlainFile =>
             // The dependency comes from a class file
             binaryDependency(pf.file, binaryClassName)

diff --git a/internal/zinc-compile-core/src/main/scala/sbt/internal/inc/CompilerArguments.scala b/internal/zinc-compile-core/src/main/scala/sbt/internal/inc/CompilerArguments.scala
@@ -82,7 +82,15 @@ final class CompilerArguments(
    * @param addLibrary Flag to return the Scala library.
    */
   def createBootClasspath(addLibrary: Boolean): String = {
-    val originalBoot = System.getProperty("sun.boot.class.path", "")
+    def findBoot: String = {
+      import scala.collection.JavaConverters._
+      System.getProperties.asScala.iterator
+        .collectFirst {
+          case (k, v) if k.endsWith(".boot.class.path") => v
+        }
+        .getOrElse("")
+    }
+    val originalBoot = Option(System.getProperty("sun.boot.class.path")).getOrElse(findBoot)
     if (addLibrary) {
       val newBootPrefix =
         if (originalBoot.isEmpty) ""
@@ -109,7 +117,8 @@ final class CompilerArguments(
     bootClasspath(hasLibrary(classpath))
 
   def extClasspath: Seq[File] =
-    (IO.parseClasspath(System.getProperty("java.ext.dirs", "")) * "*.jar").get
+    List("java.ext.dirs", "scala.ext.dirs").flatMap(k =>
+      (IO.parseClasspath(System.getProperty(k, "")) * "*.jar").get)
 
   private[this] def include(flag: Boolean, jars: File*) =
     if (flag || ScalaInstance.isDotty(scalaInstance.version)) jars