diff options
| author |  Thiago Farina <tfarina@chromium.org> | 2016-10-26 14:56:25 +0000 |
|---|---|---|
| committer |  Laszlo Csomor <laszlocsomor@google.com> | 2016-10-27 09:25:50 +0000 |
| commit | f41c252f7ed2c2590c87477f849a7e836a3867a2 (patch) | |
| tree | e1d6f29273bceb6810d44c6877c9957ef14ec78c /src/main/java/com/google/devtools/build/lib/rules/cpp/FakeCppCompileAction.java | |
| parent | 8a8a7fce075db5fd633f01f65248defdeef4e041 (diff) | |
stop passing BuildConfiguration to CppCompileAction
This fixes Ulf's TODO by adding fields for the local shell environment and code
coverage and as stated on it accessing anything other than these fields can
impact correctness.
--
Change-Id: I9ccebaa0bc8ed920e2941b3d156692cb1e0fe117
Reviewed-on: https://bazel-review.googlesource.com/c/6870/
MOS_MIGRATED_REVID=137275206
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/rules/cpp/FakeCppCompileAction.java')
| -rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/cpp/FakeCppCompileAction.java | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/FakeCppCompileAction.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/FakeCppCompileAction.java index 5bf6f94a77..4442ac2f45 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/FakeCppCompileAction.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/FakeCppCompileAction.java @@ -98,7 +98,8 @@ public class FakeCppCompileAction extends CppCompileAction { null, null, null, - configuration, + configuration.getLocalShellEnvironment(), + configuration.isCodeCoverageEnabled(), cppConfiguration, // We only allow inclusion of header files explicitly declared in // "srcs", so we only use declaredIncludeSrcs, not declaredIncludeDirs. |