From a79bff1bf45c8c89783dbcbeca0d6554135ac3b8 Mon Sep 17 00:00:00 2001 From: Ulf Adams Date: Fri, 6 Dec 2019 18:42:05 -0800 Subject: [PATCH] Delegate coverage tools to @remote_coverage_tools Requires: - #10379 is merged - a new remote coverage tools zip pushed - coverage.WORKSPACE updated to the new tools This changes the @bazel_tools//tools/test/BUILD file to fully delegate to the @remote_coverage_tools repository, which must contain rules for :lcov_merger and :coverage_report_generator. This makes the @remote_coverage_tools reference self-contained, which allows overriding the tools using --override_repository, and allows independently replacing or fixing them. Progress on #4685. Change-Id: I321c62332f00d910f4ccfb3244d63e60627d59ad --- .../build/lib/analysis/BaseRuleClasses.java | 2 +- tools/BUILD | 1 - tools/test/BUILD.tools | 9 +++++++-- .../google/devtools/coverageoutputgenerator/BUILD | 8 -------- .../devtools/coverageoutputgenerator/BUILD.tools | 14 -------------- 5 files changed, 8 insertions(+), 26 deletions(-) delete mode 100644 tools/test/CoverageOutputGenerator/java/com/google/devtools/coverageoutputgenerator/BUILD.tools diff --git a/src/main/java/com/google/devtools/build/lib/analysis/BaseRuleClasses.java b/src/main/java/com/google/devtools/build/lib/analysis/BaseRuleClasses.java index d13b0f82b60e19..18daf4dd5094f2 100644 --- a/src/main/java/com/google/devtools/build/lib/analysis/BaseRuleClasses.java +++ b/src/main/java/com/google/devtools/build/lib/analysis/BaseRuleClasses.java @@ -104,7 +104,7 @@ public static LabelLateBoundDefault coverageSupportAttribute( "//tools/test:coverage_report_generator"; private static final String DEFAULT_COVERAGE_OUTPUT_GENERATOR_VALUE = - "@bazel_tools//tools/test/CoverageOutputGenerator/java/com/google/devtools/coverageoutputgenerator:Main"; + "@bazel_tools//tools/test:lcov_merger"; @AutoCodec static final Resolver COVERAGE_REPORT_GENERATOR_CONFIGURATION_RESOLVER = diff --git a/tools/BUILD b/tools/BUILD index 15c409cfabd764..141b343ae6ad97 100644 --- a/tools/BUILD +++ b/tools/BUILD @@ -61,7 +61,6 @@ filegroup( "//tools/python:embedded_tools", "//tools/runfiles:embedded_tools", "//tools/test:embedded_tools", - "//tools/test/CoverageOutputGenerator/java/com/google/devtools/coverageoutputgenerator:embedded_tools", "//tools/osx/crosstool:srcs", "//tools/osx:srcs", "//tools/sh:embedded_tools", diff --git a/tools/test/BUILD.tools b/tools/test/BUILD.tools index e0b1186119d2ea..899258fd41ab47 100644 --- a/tools/test/BUILD.tools +++ b/tools/test/BUILD.tools @@ -33,9 +33,14 @@ filegroup( srcs = ["collect_coverage.sh"], ) -filegroup( +alias( name = "coverage_report_generator", - srcs = ["@bazel_tools//tools/test/CoverageOutputGenerator/java/com/google/devtools/coverageoutputgenerator:Main"], + actual = "@remote_coverage_tools//:coverage_report_generator", +) + +alias( + name = "lcov_merger", + actual = "@remote_coverage_tools//:lcov_merger", ) filegroup( diff --git a/tools/test/CoverageOutputGenerator/java/com/google/devtools/coverageoutputgenerator/BUILD b/tools/test/CoverageOutputGenerator/java/com/google/devtools/coverageoutputgenerator/BUILD index 70594bbb2f6b3c..1eb9c2b8eaf37e 100644 --- a/tools/test/CoverageOutputGenerator/java/com/google/devtools/coverageoutputgenerator/BUILD +++ b/tools/test/CoverageOutputGenerator/java/com/google/devtools/coverageoutputgenerator/BUILD @@ -170,11 +170,3 @@ filegroup( srcs = glob(["**"]), visibility = ["//visibility:public"], ) - -filegroup( - name = "embedded_tools", - srcs = [ - "BUILD.tools", - ], - visibility = ["//visibility:public"], -) diff --git a/tools/test/CoverageOutputGenerator/java/com/google/devtools/coverageoutputgenerator/BUILD.tools b/tools/test/CoverageOutputGenerator/java/com/google/devtools/coverageoutputgenerator/BUILD.tools deleted file mode 100644 index 262df19dda8258..00000000000000 --- a/tools/test/CoverageOutputGenerator/java/com/google/devtools/coverageoutputgenerator/BUILD.tools +++ /dev/null @@ -1,14 +0,0 @@ -package(default_visibility = ["//visibility:public"]) -load("@rules_java//java:defs.bzl", "java_binary", "java_import") - -java_import( - name = "all_lcov_merger_lib", - jars = ["@remote_coverage_tools//:all_lcov_merger_tools_deploy.jar"], -) - -java_binary( - name = "Main", - create_executable = 1, - main_class = "com.google.devtools.coverageoutputgenerator.Main", - runtime_deps = [":all_lcov_merger_lib"], -)