Remove an unnecessary addInput call in ctx.actions.run().

SpawnAction.Builder.setExecutable adds the executable to the inputs for you. Closes #5793. PiperOrigin-RevId: 210381105
bazelbuild · Aug 27, 2018 · cd57e6c · cd57e6c
1 parent 1f72416
commit cd57e6c
Showing 1 changed file with 0 additions and 1 deletion.
diff --git a/src/main/java/com/google/devtools/build/lib/analysis/skylark/SkylarkActionFactory.java b/src/main/java/com/google/devtools/build/lib/analysis/skylark/SkylarkActionFactory.java
@@ -205,7 +205,6 @@ public void run(
     buildCommandLine(builder, argumentsList);
     if (executableUnchecked instanceof Artifact) {
       Artifact executable = (Artifact) executableUnchecked;
-      builder.addInput(executable);
       FilesToRunProvider provider = context.getExecutableRunfiles(executable);
       if (provider == null) {
         builder.setExecutable(executable);