-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Include toolchain specification in error message #9625
Conversation
r? @Eh2406 (rust-highfive has picked a reviewer for you, use r? to override) |
Signed-off-by: hi-rustin <rustin.liu@gmail.com>
Should this get a test? |
Yes, I think it needs a test. But no relevant test example was found. How do you suggest to test it? |
Thanks! A test would be nice, but it would be quite difficult to test this reliably (it would need to override the sysroot with a custom one with rust-src missing). I think it is fine to go without for now. @bors r+ |
📌 Commit 00bf770 has been approved by |
☀️ Test successful - checks-actions |
Update cargo 9 commits in 9233aa06c801801cff75df65df718d70905a235e..4952979031e2cf1d901c817a32e25a156a19db4c 2021-06-22 21:32:55 +0000 to 2021-07-01 01:14:50 +0000 - Fix `BorrowMutError` when calling `cargo doc --open` (rust-lang/cargo#9531) - Exclude `target` from content-indexing on Windows (rust-lang/cargo#9635) - Temporarily ignore 2021 edition fix. (rust-lang/cargo#9642) - Temporarily disable future_incompat tests. (rust-lang/cargo#9638) - Include toolchain specification in error message (rust-lang/cargo#9625) - Error when packaging with git dependencies without version (rust-lang/cargo#9612) - simply 'if' block (rust-lang/cargo#9615) - tidy some closures and iterators (rust-lang/cargo#9614) - use 'writeln' instead of appending newline character (rust-lang/cargo#9620)
close https://github.com/rust-lang/rustup/issues/2795