Add runfiles for HOST executables for actions without commands too

6c51059e0a04d4170d10bfedbd9553b498cf28e7 adds the runfiles for HOST
executables listed as inputs to SpawnActions with a command, but it
seems reasonable to be able to execute those with non-command
SpawnActions (ie shell scripts) too.

--
Change-Id: I33b0fe3936afa87c69046350a53ee1485f2beab3
Reviewed-on: https://bazel-review.googlesource.com/#/c/2260/
MOS_MIGRATED_REVID=106845702

1 files changed, 6 insertions, 5 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/rules/SkylarkRuleImplementationFunctions.java b/src/main/java/com/google/devtools/build/lib/rules/SkylarkRuleImplementationFunctions.java
index facba30099..fda179331f 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/SkylarkRuleImplementationFunctions.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/SkylarkRuleImplementationFunctions.java
@@ -260,12 +260,13 @@ public class SkylarkRuleImplementationFunctions {
                       + "command instead of "
                       + EvalUtils.getDataTypeName(commandO));
             }
-
-            // The actual command can refer to an executable from the inputs, which could 
-            // require some runfiles. Consequently, we add the runfiles of every executable
-            // input file that is in HOST configuration to the action as a precaution.
-            addRequiredIndirectRunfiles(ctx, builder);
           }
+
+          // The actual command can refer to an executable from the inputs, which could
+          // require some runfiles. Consequently, we add the runfiles of every executable
+          // input file that is in HOST configuration to the action as a precaution.
+          addRequiredIndirectRunfiles(ctx, builder);
+
           if (mnemonicO != Runtime.NONE) {
             builder.setMnemonic((String) mnemonicO);
           }