diff options
author | 2016-06-29 09:14:10 +0000 | |
---|---|---|
committer | 2016-06-29 10:57:53 +0000 | |
commit | 8f460f399123709d5b07c68184147ab024e1c593 (patch) | |
tree | 39bee12ac3024e653c1bd30ffe39cb5d25439750 /src/main/java/com/google/devtools/build/lib/rules/objc/TestSupport.java | |
parent | 02f2348c72f07fc4df4fbc83a86725c0e5eb9146 (diff) |
Refactor how coverage support files get to test actions.
Previously we used labels in each configuration fragment that then got added to every test action. Instead, we now have a filegroup under //tools/test for coverage files that truly need to be on the inputs of every test action and collect language-specific support files in InstrumentedFilesProvider.
This makes configuration creation simpler and makes it possible to turn --crosstool_top into something else other than a filegroup (previously, it was that filegroup that got added to every test action)
--
MOS_MIGRATED_REVID=126170241
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/rules/objc/TestSupport.java')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/objc/TestSupport.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/objc/TestSupport.java b/src/main/java/com/google/devtools/build/lib/rules/objc/TestSupport.java index 7b911c5866..e813e19319 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/objc/TestSupport.java +++ b/src/main/java/com/google/devtools/build/lib/rules/objc/TestSupport.java @@ -265,7 +265,7 @@ public class TestSupport { if (ruleContext.getConfiguration().isCodeCoverageEnabled()) { envBuilder.put("COVERAGE_GCOV_PATH", - ruleContext.getHostPrerequisiteArtifact(IosTest.GCOV_ATTR).getExecPathString()); + ruleContext.getHostPrerequisiteArtifact(IosTest.OBJC_GCOV_ATTR).getExecPathString()); envBuilder.put("APPLE_COVERAGE", "1"); } |