Skip to content

Commit

Permalink
Restrict unecessary inputs to 'JdepsMerge' actions (#1175)
Browse files Browse the repository at this point in the history
  • Loading branch information
oliviernotteghem committed May 24, 2024
1 parent 6f3c7db commit 257b9ad
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions kotlin/internal/jvm/compile.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -346,8 +346,10 @@ def _run_merge_jdeps_action(ctx, toolchains, jdeps, outputs, deps):
len(jdeps),
)

# For sandboxing to work, and for this action to be deterministic, the compile jars need to be passed as inputs
inputs = depset(jdeps, transitive = [depset([], transitive = [dep.transitive_compile_time_jars for dep in deps])])
inputs = depset(jdeps)
if not toolchains.kt.experimental_report_unused_deps == "off":
# For sandboxing to work, and for this action to be deterministic, the compile jars need to be passed as inputs
inputs = depset(jdeps, transitive = [depset([], transitive = [dep.transitive_compile_time_jars for dep in deps])])

ctx.actions.run(
mnemonic = mnemonic,
Expand Down

0 comments on commit 257b9ad

Please sign in to comment.