Skip to content

Commit

Permalink
Remove the py_builtins.new_runfiles_supplier method.
Browse files Browse the repository at this point in the history
It does not have any hits on GitHub or on public Google search.

RELNOTES: None.
PiperOrigin-RevId: 595095994
Change-Id: Ic435e067b2c4ca82eebacaaf5f5f76b239b5dd62
  • Loading branch information
lberki authored and copybara-github committed Jan 2, 2024
1 parent 40271d7 commit 9790f6d
Showing 1 changed file with 0 additions and 20 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,6 @@
import com.google.devtools.build.lib.analysis.FilesToRunProvider;
import com.google.devtools.build.lib.analysis.RepoMappingManifestAction;
import com.google.devtools.build.lib.analysis.Runfiles;
import com.google.devtools.build.lib.analysis.SingleRunfilesSupplier;
import com.google.devtools.build.lib.analysis.SourceManifestAction;
import com.google.devtools.build.lib.analysis.SourceManifestAction.ManifestType;
import com.google.devtools.build.lib.analysis.TransitiveInfoCollection;
Expand Down Expand Up @@ -135,25 +134,6 @@ public String getCurrentOsName() {
return OS.getCurrent().getCanonicalName();
}

// TODO(b/69113360): Remove once par-generation is moved out of the py_binary rule itself.
@StarlarkMethod(
name = "new_runfiles_supplier",
doc = "Create a RunfilesSupplier, which can be passed to ctx.actions.run.input_manifests.",
parameters = {
@Param(name = "ctx", positional = false, named = true, defaultValue = "unbound"),
@Param(name = "runfiles_dir", positional = false, named = true, defaultValue = "unbound"),
@Param(name = "runfiles", positional = false, named = true, defaultValue = "unbound"),
})
public Object addEnv(StarlarkRuleContext ruleContext, String runfilesStr, Runfiles runfiles)
throws EvalException {
return new SingleRunfilesSupplier(
PathFragment.create(runfilesStr),
runfiles,
/* repoMappingManifest= */ null,
ruleContext.getConfiguration().getRunfileSymlinksMode(),
ruleContext.getConfiguration().buildRunfileLinks());
}

// TODO(rlevasseur): Remove once Starlark exposes this directly, see
// https://github.com/bazelbuild/bazel/issues/15164
@StarlarkMethod(
Expand Down

0 comments on commit 9790f6d

Please sign in to comment.