commit fbf2756c649649bc821595c562721ebf1af10361
author ulfjack <ulfjack@google.com> Fri May 18 12:46:54 2018 -0700
committer Copybara-Service <copybara-piper@google.com> Fri May 18 12:48:25 2018 -0700
tree f1251754c292c922f421a9368c01841232d322ad
parent beaaf669b6f3651445da222cf43b6c33546c0fb7

Add BuildEventProtocolOptions; use to enable/disable important_outputs

The important_outputs field is deprecated, and this adds a flag to disable
its generation entirely.

PiperOrigin-RevId: 197186530

src/main/java/com/google/devtools/build/lib/analysis/TargetCompleteEvent.java

diff --git a/src/main/java/com/google/devtools/build/lib/analysis/TargetCompleteEvent.java b/src/main/java/com/google/devtools/build/lib/analysis/TargetCompleteEvent.java
index e6f5c3f..a359446 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/TargetCompleteEvent.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/TargetCompleteEvent.java
@@ -232,10 +232,12 @@
 
     // TODO(aehlig): remove direct reporting of artifacts as soon as clients no longer
     // need it.
-    addImportantOutputs(builder, converters, getLegacyFilteredImportantArtifacts());
-    if (baselineCoverageArtifacts != null) {
-      addImportantOutputs(
-          builder, (artifact -> BASELINE_COVERAGE), converters, baselineCoverageArtifacts);
+    if (converters.getOptions().legacyImportantOutputs) {
+      addImportantOutputs(builder, converters, getLegacyFilteredImportantArtifacts());
+      if (baselineCoverageArtifacts != null) {
+        addImportantOutputs(
+            builder, (artifact -> BASELINE_COVERAGE), converters, baselineCoverageArtifacts);
+      }
     }
 
     BuildEventStreamProtos.TargetComplete complete = builder.build();