Skip to content

fix: reconcile toolchain declarations across modules - #259

Open
dzbarsky wants to merge 1 commit into
mainfrom
codex/reconcile-toolchain-tags
Open

dzbarsky wants to merge 1 commit into
mainfrom
codex/reconcile-toolchain-tags

Conversation

@dzbarsky

Copy link
Copy Markdown
Member

When multiple modules declare the same toolchain repository, use the root module's complete configuration if present. Otherwise, independently select the highest Rust version, edition, rustfmt version, and rust-analyzer version. Resolve configurations before downloading archives so overridden dependency versions are ignored. Keep actionable errors for conflicting compiler flags and mixed stable/beta/nightly channels, and preserve normal and dev dependency metadata for tagless default-toolchain imports.

Validation: all 11 //rs/toolchains:toolchain_config_tests tests pass; five actual Bazel module configurations verify dependency maxima, root overrides, normal/dev imports, and no-tag defaults. Buildifier formatting and git diff --check pass.

Fixes #256

-zbarskybot

Prefer root-module configurations for each toolchain repository. Otherwise select compiler, edition, rustfmt, and rust-analyzer maxima before declaring repositories or downloading archives. Preserve errors for conflicting compiler flags and incomparable release channels.

Keep original root tags for extension dependency metadata and preserve tagless default-toolchain imports.

Fixes #256
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.

Conflicting default_toolchain specification from multiple modules

1 participant