Do not filter debug messages

Debug messages (generated with Skylark's `print` function) used to be filtered out by the output filter: if such messages are generated during the analysis phase in a package different to the target package (e.g. if a user builds //foo:foo and the message is generated in a dependency //bar:bar), the message are not shown by default, which makes an erroneous impression that the code for //bar:bar hasn't been executed at all and interferes with debugging. Now the output filter doesn't affect debug messages at all.

RELNOTES: Debug messages generated by `print()` are not being filtered out by --output_filter anymore, it's recommended not to use them in production code.
PiperOrigin-RevId: 177431255

1 files changed, 5 insertions, 2 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/events/Reporter.java b/src/main/java/com/google/devtools/build/lib/events/Reporter.java
index 6c164a1ebd..e654748422 100644
--- a/src/main/java/com/google/devtools/build/lib/events/Reporter.java
+++ b/src/main/java/com/google/devtools/build/lib/events/Reporter.java
@@ -70,7 +70,7 @@ public final class Reporter implements ExtendedEventHandler, ExceptionListener {
   /** Constructor which configures a reporter with the specified handlers. */
   public Reporter(EventBus eventBus, EventHandler... handlers) {
     this.eventBus = eventBus;
-    for (EventHandler handler: handlers) {
+    for (EventHandler handler : handlers) {
       addHandler(handler);
     }
   }
@@ -103,7 +103,10 @@ public final class Reporter implements ExtendedEventHandler, ExceptionListener {
    */
   @Override
   public synchronized void handle(Event e) {
-    if (e.getKind() != EventKind.ERROR && e.getTag() != null && !showOutput(e.getTag())) {
+    if (e.getKind() != EventKind.ERROR
+        && e.getKind() != EventKind.DEBUG
+        && e.getTag() != null
+        && !showOutput(e.getTag())) {
       return;
     }
     for (EventHandler handler : handlers) {