Skip to content

Reduce //cargo public interface - #4198

Open
UebelAndre wants to merge 2 commits into
bazelbuild:mainfrom
UebelAndre:cargo_incompat
Open

Reduce //cargo public interface#4198
UebelAndre wants to merge 2 commits into
bazelbuild:mainfrom
UebelAndre:cargo_incompat

Conversation

@UebelAndre

Copy link
Copy Markdown
Collaborator

Rules dropped:

  • cargo_bootstrap_repository: Building rust code in a repository rule is a massive anti-pattern and is incredibly slow. It's only intended use is internal for crate_universe testing.
  • cargo_dep_env: This rule has a confusing interface and otherwise seems unmaintained.

@UebelAndre
UebelAndre marked this pull request as ready for review August 3, 2026 14:59
@UebelAndre

Copy link
Copy Markdown
Collaborator Author

@krasimirgg seems like you were the last to update cargo_dep_env (#4123)? What is this rule for? It seems like it turns a directory into the output of a build script? Why not have users just fill out the provider on their custom rule?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant