commit 39e5a46294de31baa0411f6a59119427c66305da
author Damien Martin-Guillerez <dmarting@google.com> Tue Dec 20 10:58:17 2016 +0000
committer Klaus Aehlig <aehlig@google.com> Tue Dec 20 11:07:10 2016 +0000
tree 90aaf7e33e5e547508e032a6c652635daf7164b1
parent d40c78a98e2b1f562170396f94bf54d17c6958d0

Fix cycle in dependency graph during bootstrap

This cycle was due to JacocoCoverage_deploy.jar being an implicit dependency
of java_binary. We do not need the binary during bootstrap because we are not
running with coverage but we need the target. Simply move it to exists as a
java_import in tools/jdk.

Fixes #2269.
To be cherry-picked for #2246.

--
Change-Id: I63aee0e76b305833ac0c0c2cae5abe1cbad5e6d1
Reviewed-on: https://cr.bazel.build/8036
PiperOrigin-RevId: 142537244
MOS_MIGRATED_REVID=142537244

tools/jdk/BUILD

diff --git a/tools/jdk/BUILD b/tools/jdk/BUILD
index 5b5596b..badd086 100644
--- a/tools/jdk/BUILD
+++ b/tools/jdk/BUILD
@@ -162,7 +162,7 @@
     actual = "//third_party/java/jacoco:blaze-agent",
 )
 
-alias(
+java_import(
     name = "JacocoCoverage",
-    actual = "//src/java_tools/junitrunner/java/com/google/testing/coverage:JacocoCoverage",
+    jars = [":JacocoCoverage_deploy.jar"],
 )