Skip to content

Commit

Permalink
fix: Add tool runfiles to action
Browse files Browse the repository at this point in the history
Previously, tool runfiles were not present at execution time. For
example, if the provided tool was a py_binary, the py_binary would fail
to run due to the omission of the runfiles.

This relates to bazelbuild/bazel#10110
  • Loading branch information
jheaff1 committed Jul 19, 2022
1 parent 2e88701 commit ac9ea33
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion foreign_cc/private/framework.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -473,7 +473,8 @@ def cc_external_rule_impl(ctx, attrs):
ctx.files.build_data +
legacy_tools +
[cc_toolchain.all_files] +
[data[DefaultInfo].default_runfiles.files for data in data_dependencies],
[data[DefaultInfo].default_runfiles.files for data in data_dependencies] +
[data[DefaultInfo].files_to_run for data in data_dependencies],
command = wrapped_outputs.wrapper_script_file.path,
execution_requirements = execution_requirements,
use_default_shell_env = True,
Expand Down

0 comments on commit ac9ea33

Please sign in to comment.