diff options
author | 2017-04-27 18:42:18 +0200 | |
---|---|---|
committer | 2017-04-28 01:01:39 +0200 | |
commit | 4685557d45c11d9b9422418001b8c457bbb83624 (patch) | |
tree | c86c76e17df764e65f2bca2b9ee1925c14b06eb4 /src/main | |
parent | 7dc34167d797af692add7c87e6e14208efada585 (diff) |
Re-introduce the overall_success field in BEP.
Removal of BuildFinished.overall_success broke internal tests and thus we
decided to reintroduce the field and deprecate it.
RELNOTES: None.
PiperOrigin-RevId: 154432961
Diffstat (limited to 'src/main')
2 files changed, 3 insertions, 1 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/buildeventstream/proto/build_event_stream.proto b/src/main/java/com/google/devtools/build/lib/buildeventstream/proto/build_event_stream.proto index dca2dc7679..e21a8494e1 100644 --- a/src/main/java/com/google/devtools/build/lib/buildeventstream/proto/build_event_stream.proto +++ b/src/main/java/com/google/devtools/build/lib/buildeventstream/proto/build_event_stream.proto @@ -379,7 +379,8 @@ message BuildFinished { int32 code = 2; } - reserved 1; + // If the build succeeded or failed. + bool overall_success = 1 [deprecated = true]; // The overall status of the build. A build was successfull iff // ExitCode.code equals 0. diff --git a/src/main/java/com/google/devtools/build/lib/buildtool/buildevent/BuildCompleteEvent.java b/src/main/java/com/google/devtools/build/lib/buildtool/buildevent/BuildCompleteEvent.java index 936aeb12a5..45b4ed54d7 100644 --- a/src/main/java/com/google/devtools/build/lib/buildtool/buildevent/BuildCompleteEvent.java +++ b/src/main/java/com/google/devtools/build/lib/buildtool/buildevent/BuildCompleteEvent.java @@ -68,6 +68,7 @@ public final class BuildCompleteEvent implements BuildEvent { BuildFinished finished = BuildFinished.newBuilder() + .setOverallSuccess(result.getSuccess()) .setExitCode(exitCode) .setFinishTimeMillis(result.getStopTime()) .build(); |