[7.1.1] Actually use shouldPublish() to determine whether to publish …

…the execution log to the BEP. (#21671) This was the original intent in 9eb3b0a. PiperOrigin-RevId: 614986613 Change-Id: I356788d82a6d36a7a465a1bb828e4074aad11385
bazelbuild · Mar 13, 2024 · e67c6cc · e67c6cc
1 parent d7be1d9
commit e67c6cc
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/main/java/com/google/devtools/build/lib/bazel/SpawnLogModule.java b/src/main/java/com/google/devtools/build/lib/bazel/SpawnLogModule.java
@@ -181,7 +181,9 @@ public void buildComplete(BuildCompleteEvent event) {
 
     try {
       spawnLogContext.close();
-      event.getResult().getBuildToolLogCollection().addLocalFile("execution.log", outputPath);
+      if (spawnLogContext.shouldPublish()) {
+        event.getResult().getBuildToolLogCollection().addLocalFile("execution.log", outputPath);
+      }
     } catch (IOException e) {
       abruptExit =
           new AbruptExitException(