commit e414d6f2135611240c6a7991bf7587c48ba0c68e
author messa <messa@google.com> Tue Apr 06 03:12:15 2021 -0700
committer Copybara-Service <copybara-worker@google.com> Tue Apr 06 03:13:15 2021 -0700
tree 9327f8c5f2932ca43e8cbb9ba4c6791635d8267f
parent 36116ce73f97a7d2ac528dca07cc77bc772c9a22

Fix comment in Starlark action.

The Starlark action no longer depends on its shadowed action's outputs.

PiperOrigin-RevId: 366970303

src/main/java/com/google/devtools/build/lib/analysis/actions/StarlarkAction.java

diff --git a/src/main/java/com/google/devtools/build/lib/analysis/actions/StarlarkAction.java b/src/main/java/com/google/devtools/build/lib/analysis/actions/StarlarkAction.java
index c43e866..f1a6dee 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/actions/StarlarkAction.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/actions/StarlarkAction.java
@@ -168,8 +168,7 @@
   public NestedSet<Artifact> discoverInputs(ActionExecutionContext actionExecutionContext)
       throws ActionExecutionException, InterruptedException {
     // If the Starlark action shadows another action and the shadowed action discovers its inputs,
-    // we depend on the outputs of the action doing input discovery and it should know its inputs
-    // after having been executed.
+    // we get the shadowed action's discovered inputs and append it to the Starlark action inputs.
     if (shadowedAction.isPresent() && shadowedAction.get().discoversInputs()) {
       Action shadowedActionObj = shadowedAction.get();