tree 8bc59a4085d8ac1044b5d3ed391df5c9ff87db9e
parent 4a8f298afefbb61b534bb950116fa71c489c412a
author John Millikin <john@john-millikin.com> 1573647527 -0800
committer Philipp Wollermann <philwo@google.com> 1574241928 +0100

Add tool executables (from FilesToRunProvider) to action inputs.

Fixes https://github.com/bazelbuild/bazel/issues/7390

Summary: rules are allowed to set `DefaultInfo(files = depset(), executable = ...)`. If the resulting targets are used as tool inputs to another target, the executable will not be added to the downstream targets' action inputs. This manifests as either an `execvp()` error (from `ctx.actions.run()`) or a Bash error in genrules.

This PR fixes that so the use case of non-default executables works as expected. This is useful for building scripts with complex runfiles, where `bazel run //:something` and `bazel build //:something && bazel-bin/something` cannot be supported by the same output file.

cc @ccate @jmmv

Closes #10110.

PiperOrigin-RevId: 280170524