diff options
author | Klaus Aehlig <aehlig@google.com> | 2016-10-26 14:27:48 +0000 |
---|---|---|
committer | Laszlo Csomor <laszlocsomor@google.com> | 2016-10-27 09:24:43 +0000 |
commit | 8a8a7fce075db5fd633f01f65248defdeef4e041 (patch) | |
tree | 612178ae9da53978d063a8617cb75398bea7480d /src/main/java/com/google/devtools/build/lib/BUILD | |
parent | 4db50640e3799c528b1c893c431227d5d5c192a3 (diff) |
Report completion of a target together with failed actions
Report the completion of all targets together with the root causes on the build
event stream. To do so, have TargetCompleteEvent and ActionExecutedEvent be
instances of BuildEvent; however, ignore an ActionExecutedEvent in the
BuildEventStreamer if the execution was successful.
By this change we get, for the first time, a build event stream that is naturally
closed, i.e., without Aborted events closing up lose ends. Add a test asserting
this property.
--
Change-Id: Ie90dd52ee80deb0fdabdce1da551935522880a1a
Reviewed-on: https://bazel-review.googlesource.com/#/c/6279
MOS_MIGRATED_REVID=137273002
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/BUILD')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/BUILD | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/BUILD b/src/main/java/com/google/devtools/build/lib/BUILD index 0478519a09..1d3dcc97eb 100644 --- a/src/main/java/com/google/devtools/build/lib/BUILD +++ b/src/main/java/com/google/devtools/build/lib/BUILD @@ -304,6 +304,7 @@ java_library( srcs = glob(["buildeventstream/*.java"]), deps = [ "//src/main/java/com/google/devtools/build/lib/buildeventstream/proto:build_event_stream_java_proto", + "//src/main/java/com/google/devtools/build/lib/causes", "//src/main/java/com/google/devtools/build/lib/cmdline", "//third_party:guava", "//third_party:jsr305", @@ -504,6 +505,7 @@ java_library( ], deps = [ ":base-util", + ":buildeventstream", ":collect", ":concurrent", ":events", @@ -521,6 +523,7 @@ java_library( ":util", ":vfs", "//src/main/java/com/google/devtools/build/lib/actions", + "//src/main/java/com/google/devtools/build/lib/buildeventstream/proto:build_event_stream_java_proto", "//src/main/java/com/google/devtools/build/lib/causes", "//src/main/java/com/google/devtools/build/lib/cmdline", "//src/main/java/com/google/devtools/build/skyframe", |