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

Include toolchain specification in error message #9625

Merged
merged 1 commit into from
Jun 25, 2021

Conversation

Rustin170506
Copy link
Member

@rust-highfive
Copy link

r? @Eh2406

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 25, 2021
Signed-off-by: hi-rustin <rustin.liu@gmail.com>
@Eh2406
Copy link
Contributor

Eh2406 commented Jun 25, 2021

Should this get a test?

@Rustin170506
Copy link
Member Author

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?

@ehuss
Copy link
Contributor

ehuss commented Jun 25, 2021

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+

@bors
Copy link
Contributor

bors commented Jun 25, 2021

📌 Commit 00bf770 has been approved by ehuss

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jun 25, 2021
@bors
Copy link
Contributor

bors commented Jun 25, 2021

⌛ Testing commit 00bf770 with merge 3691752...

@bors
Copy link
Contributor

bors commented Jun 25, 2021

☀️ Test successful - checks-actions
Approved by: ehuss
Pushing 3691752 to master...

@bors bors merged commit 3691752 into rust-lang:master Jun 25, 2021
@Rustin170506 Rustin170506 deleted the rustin-msg branch June 26, 2021 02:44
bors added a commit to rust-lang-ci/rust that referenced this pull request Jul 1, 2021
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)
@ehuss ehuss added this to the 1.55.0 milestone Feb 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants