Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated crate_universe bzlmod to support skipping splicing #3148

Merged
merged 6 commits into from
Jan 8, 2025

Conversation

UebelAndre
Copy link
Collaborator

Additional Changes:

  • Updated crate_universe macros to refer to aliases instead of direct external target for bzlmod support.
  • Added rust_host_tools_nightly module for exposing a nightly toolchain
  • Converted examples/crate_universe to use bzlmod

@UebelAndre UebelAndre force-pushed the examples_crate_universe branch 15 times, most recently from f25503b to ff54164 Compare January 2, 2025 04:15
@UebelAndre UebelAndre marked this pull request as ready for review January 2, 2025 05:30
@UebelAndre UebelAndre force-pushed the examples_crate_universe branch from ff54164 to 94f9b31 Compare January 3, 2025 01:25
@UebelAndre UebelAndre force-pushed the examples_crate_universe branch from 94f9b31 to 7b279b9 Compare January 3, 2025 03:40
@UebelAndre UebelAndre added this pull request to the merge queue Jan 8, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 8, 2025
@UebelAndre UebelAndre added this pull request to the merge queue Jan 8, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 8, 2025
@UebelAndre UebelAndre added this pull request to the merge queue Jan 8, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 8, 2025
@UebelAndre UebelAndre enabled auto-merge January 8, 2025 01:03
@UebelAndre UebelAndre added this pull request to the merge queue Jan 8, 2025
Merged via the queue into bazelbuild:main with commit 5e90483 Jan 8, 2025
3 checks passed
@UebelAndre UebelAndre deleted the examples_crate_universe branch January 8, 2025 01:20
hauserx added a commit to hauserx/rules_rust that referenced this pull request Jan 20, 2025
The bazel-lsp uses `rust_host_tools` to perform some actions:
by using:

rust_host_tools = use_extension("@rules_rust//rust:extensions.bzl", "rust_host_tools")
use_repo(rust_host_tools, "rust_host_tools")

See: https://github.com/cameron-martin/bazel-lsp/blob/19b799e528c884be6efeb108ed1840a4edff1330/MODULE.bazel#L14

It started failing after
bazelbuild#3148 because rules_rust's
MODULE.bazel registers toolchain with the same name:

rust_host_tools.host_tools(
    name = "rust_host_tools",
)

See: https://github.com/bazelbuild/rules_rust/blob/aae84e97c73eae2e6654e1a7e1b751d0c1f2ac9e/MODULE.bazel#L70
hauserx added a commit to hauserx/rules_rust that referenced this pull request Jan 20, 2025
The bazel-lsp uses `rust_host_tools` to perform some actions:
by using:

rust_host_tools = use_extension("@rules_rust//rust:extensions.bzl", "rust_host_tools")
use_repo(rust_host_tools, "rust_host_tools")

See: https://github.com/cameron-martin/bazel-lsp/blob/19b799e528c884be6efeb108ed1840a4edff1330/MODULE.bazel#L14

It started failing after
bazelbuild#3148 because rules_rust's
MODULE.bazel registers toolchain with the same name:

rust_host_tools.host_tools(
    name = "rust_host_tools",
)

See: https://github.com/bazelbuild/rules_rust/blob/aae84e97c73eae2e6654e1a7e1b751d0c1f2ac9e/MODULE.bazel#L70
github-merge-queue bot pushed a commit that referenced this pull request Jan 20, 2025
The bazel-lsp uses `rust_host_tools` to perform some actions by using:

```starlark
rust_host_tools = use_extension("@rules_rust//rust:extensions.bzl", "rust_host_tools") 
use_repo(rust_host_tools, "rust_host_tools")
```

See:
https://github.com/cameron-martin/bazel-lsp/blob/19b799e528c884be6efeb108ed1840a4edff1330/MODULE.bazel#L14

It started failing when using rules_rust from HEAD after
#3148 because rules_rust's
MODULE.bazel registers toolchain with the same name:

```starlark
rust_host_tools.host_tools(
    name = "rust_host_tools",
)
```

See:
https://github.com/bazelbuild/rules_rust/blob/aae84e97c73eae2e6654e1a7e1b751d0c1f2ac9e/MODULE.bazel#L70
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.

2 participants