Merge pull request scala#6800 from dwijnand/add-testJDeps

Enforce library/reflect restriction to compat1 profile

Author: dwijnand

build.sbt

@@ -805,6 +805,8 @@ lazy val root: Project = (project in file("."))
       state
     },
 
+    testJDeps := TestJDeps.testJDepsImpl.value,
+
     testAll := {
       val results = ScriptCommands.sequence[(Result[Unit], String)](List(
         (Keys.test in Test in junit).result map (_ -> "junit/test"),
@@ -819,6 +821,7 @@ lazy val root: Project = (project in file("."))
         (Keys.test in Test in osgiTestEclipse).result map (_ -> "osgiTestEclipse/test"),
         (mimaReportBinaryIssues in library).result map (_ -> "library/mimaReportBinaryIssues"),
         (mimaReportBinaryIssues in reflect).result map (_ -> "reflect/mimaReportBinaryIssues"),
+        testJDeps.result map (_ -> "testJDeps"),
         (compile in Compile in bench).map(_ => ()).result map (_ -> "bench/compile"),
         Def.task(()).dependsOn( // Run these in parallel:
           doc in Compile in library,
@@ -939,6 +942,8 @@ lazy val mkQuick = taskKey[File]("Generate a full build, including scripts, in b
 lazy val mkPack = taskKey[File]("Generate a full build, including scripts, in build/pack")
 lazy val testAll = taskKey[Unit]("Run all test tasks sequentially")
 
+val testJDeps = taskKey[Unit]("Run jdeps to check dependencies")
+
 // Defining these settings is somewhat redundant as we also redefine settings that depend on them.
 // However, IntelliJ's project import works better when these are set correctly.
 def clearSourceAndResourceDirectories = Seq(Compile, Test).flatMap(config => inConfig(config)(Seq(

project/TestJDeps.scala

@@ -0,0 +1,21 @@
+package scala.build
+
+import sbt._, Keys._
+
+object TestJDeps {
+  val testJDepsImpl: Def.Initialize[Task[Unit]] = Def.task {
+    val libraryJar = (packageBin in Compile in LocalProject("library")).value
+    val reflectJar = (packageBin in Compile in LocalProject("reflect")).value
+
+    // jdeps -s -P build/pack/lib/scala-{library,reflect}.jar | grep -v build/pack | perl -pe 's/.*\((.*)\)$/$1/' | sort -u
+    val jdepsOut = Process("jdeps", Seq("-s", "-P", libraryJar.getPath, reflectJar.getPath)).lines
+
+    val profilePart = ".*\\((.*)\\)$".r
+    val profiles = jdepsOut.collect {
+      case profilePart(profile) => profile
+    }.toSet
+
+    if (profiles != Set("compact1"))
+      throw new RuntimeException(jdepsOut.mkString("Detected dependency outside of compact1:\n", "\n", ""))
+  }
+}