diff options
| author |  hlopko <hlopko@google.com> | 2018-04-24 04:07:33 -0700 |
|---|---|---|
| committer |  Copybara-Service <copybara-piper@google.com> | 2018-04-24 04:09:09 -0700 |
| commit | 8b33784fb7e0d487908aeb28c398b5e9b4078a52 (patch) | |
| tree | b0e65c3d5f38d8182c3ab337c993be869fb68a28 /src/test/java/com/google/devtools/build | |
| parent | fe935cd1b3aedd387260623fc752ba99d1348fe7 (diff) | |
Properly report completion of shared actions with input discovery
Currently we report "Analyzing" when include scanning runs. But since we can
have shared C++ compile actions, only one of the group will be executed and only
one will be reported completed. Remaining shared actions currently stay with
"analyzing" forever.
This cl makes sure that these actions are properly handled when finished.
This is an encore of https://github.com/bazelbuild/bazel/commit/24f19ec2679dd93b1ac5b06e46f3b35807d6e217. In this incarnation I make sure that all actions that discover inputs are consistent in reporting their Analyzing status. Originally only CppCompileAction was doing that. Apparently we have more actions that discover inputs (e.g. LtoBackendAction) but these were not reporting Analyzing and therefore crashing on preconditions. This cl makes sure that all actions discovering inputs report their analyzing status.
RELNOTES: None
PiperOrigin-RevId: 194066513
Diffstat (limited to 'src/test/java/com/google/devtools/build')
| -rw-r--r-- | src/test/java/com/google/devtools/build/lib/actions/util/DummyExecutor.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/actions/util/DummyExecutor.java b/src/test/java/com/google/devtools/build/lib/actions/util/DummyExecutor.java index b4f878604b..14076b606a 100644 --- a/src/test/java/com/google/devtools/build/lib/actions/util/DummyExecutor.java +++ b/src/test/java/com/google/devtools/build/lib/actions/util/DummyExecutor.java @@ -65,7 +65,7 @@ public final class DummyExecutor implements Executor { @Override public EventBus getEventBus() { - throw new UnsupportedOperationException(); + return new EventBus(); } @Override |