rust-embedded / cargo-binutils

Cargo subcommands to invoke the LLVM tools shipped with the Rust toolchain
Apache License 2.0
491 stars 43 forks source link

Component is now named 'llvm-tools' #140

Closed mbrossard closed 9 months ago

mbrossard commented 9 months ago

I don't know exactly when the name changed from llvm-tools-preview to llvm-tools but this works for 1.70 (MSRV for this crate) and later.

burrbull commented 9 months ago

rustfmt, please

mbrossard commented 9 months ago

It seems that it might be an issue in Rust (see: https://github.com/rust-lang/rustup/pull/3578). Maybe this should be reverted.

Emilgardis commented 9 months ago

we could revert, or introduce a check with rustup component check and present that name, and keep -preview in the docs for now

mbrossard commented 9 months ago

After raising the issue in Zulip, https://github.com/rust-lang/rust/issues/119164 was opened.