Skip to content

Commit d981709

Browse files
committed
Fix tools with remote exec
Previously the shared libs that are part of implicit_deps were not uploaded to remote exec spawns. I'm still confused about which one we "should" be using here, but this seems to work.
1 parent 87f860f commit d981709

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

mojo/toolchain.bzl

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,10 @@ load("@rules_cc//cc/common:cc_info.bzl", "CcInfo")
44
load("//mojo:providers.bzl", "MojoInfo", "MojoToolchainInfo")
55

66
def _mojo_toolchain_impl(ctx):
7-
tool_files = [ctx.attr.mojo[DefaultInfo].files]
8-
for dep in ctx.attr.implicit_deps + ctx.attr.extra_tools + [ctx.attr.lld]:
7+
tool_files = []
8+
for dep in ctx.attr.implicit_deps + ctx.attr.extra_tools + [ctx.attr.lld, ctx.attr.mojo]:
99
tool_files.append(dep[DefaultInfo].default_runfiles.files)
10-
tool_files.append(dep[DefaultInfo].files_to_run)
10+
tool_files.append(dep[DefaultInfo].files)
1111

1212
copts = list(ctx.attr.copts)
1313
gpu_toolchain = ctx.toolchains["//:gpu_toolchain_type"]

0 commit comments

Comments
 (0)