From 9aa308e1ef0f0f0543e3d58f3b3c5488c087c8c1 Mon Sep 17 00:00:00 2001 From: Laurent Le Brun Date: Thu, 23 May 2019 20:07:27 +0200 Subject: [PATCH] Fix repository for compatibility with --incompatible_no_support_tools_in_action_inputs (#156) --- rules/private/copy_file_private.bzl | 2 +- rules/run_binary.bzl | 5 +++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/rules/private/copy_file_private.bzl b/rules/private/copy_file_private.bzl index c39459c..b531815 100644 --- a/rules/private/copy_file_private.bzl +++ b/rules/private/copy_file_private.bzl @@ -48,7 +48,7 @@ def copy_cmd(ctx, src, dst): def copy_bash(ctx, src, dst): ctx.actions.run_shell( - inputs = [src], + tools = [src], outputs = [dst], command = "cp -f \"$1\" \"$2\"", arguments = [src.path, dst.path], diff --git a/rules/run_binary.bzl b/rules/run_binary.bzl index 76a5393..5c7fe68 100644 --- a/rules/run_binary.bzl +++ b/rules/run_binary.bzl @@ -23,7 +23,7 @@ load("//lib:dicts.bzl", "dicts") def _impl(ctx): tool_as_list = [ctx.attr.tool] tool_inputs, tool_input_mfs = ctx.resolve_tools(tools = tool_as_list) - args = [ + args = [ # Expand $(location) / $(locations) in args. # # To keep the rule simple, do not expand Make Variables (like *_binary.args usually would). @@ -44,7 +44,8 @@ def _impl(ctx): } ctx.actions.run( outputs = ctx.outputs.outs, - inputs = depset(direct = ctx.files.srcs, transitive = [tool_inputs]), + inputs = ctx.files.srcs, + tools = tool_inputs, executable = ctx.executable.tool, arguments = args, mnemonic = "RunBinary",