From a2d3aa66b2bb1f734cd33eb1f6b24cb02157b44b Mon Sep 17 00:00:00 2001 From: Keith Smiley Date: Thu, 7 Jul 2022 17:40:49 -0700 Subject: [PATCH] Remove LCOV_MERGER We now set the related attribute instead https://github.com/bazelbuild/rules_apple/pull/1388 --- .bazelrc | 4 ---- test/apple_shell_testutils.sh | 2 -- 2 files changed, 6 deletions(-) diff --git a/.bazelrc b/.bazelrc index ea9cb0d835..e2c93d13a5 100644 --- a/.bazelrc +++ b/.bazelrc @@ -17,9 +17,5 @@ test --build_tests_only # visibility and symbol import issues are resolved. build --nocheck_visibility -# Required to avoid missing coverage runner file errors. -# GitHub Issue: https://github.com/bazelbuild/rules_apple/issues/1128 -coverage --action_env=LCOV_MERGER=/usr/bin/true - # Use llvm-cov instead of gcov (default). coverage --experimental_use_llvm_covmap diff --git a/test/apple_shell_testutils.sh b/test/apple_shell_testutils.sh index 92c158f622..833dd22c5a 100755 --- a/test/apple_shell_testutils.sh +++ b/test/apple_shell_testutils.sh @@ -449,8 +449,6 @@ function do_action() { # Explicitly pass these flags to ensure the external testing infrastructure # matches the internal one. "--incompatible_merge_genfiles_directory" - # TODO: Remove this once we can use the late bound coverage attribute - "--test_env=LCOV_MERGER=/usr/bin/true" ) local bazel_version="$(bazel --version)"