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

Add more information to the error code for 'crate not found' #81676

Merged
merged 1 commit into from
Feb 5, 2021

Conversation

jyn514
Copy link
Member

@jyn514 jyn514 commented Feb 2, 2021

This comes up a lot when bootstrapping.

@jyn514 jyn514 added the A-error-codes Area: Explanation of an error code (--explain) label Feb 2, 2021
@rust-highfive
Copy link
Collaborator

Updates src/tools/cargo.

cc @ehuss

@rust-highfive
Copy link
Collaborator

r? @oli-obk

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

@rust-highfive
Copy link
Collaborator

⚠️ Warning ⚠️

  • These commits modify submodules.

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Feb 2, 2021
@poliorcetics
Copy link
Contributor

Awesome !

Comment on lines +31 to +34
- You are developing the compiler itself and haven't built libstd from source.
You can usually build it with `x.py build library/std`. More information
about x.py is available in the [rustc-dev-guide].

[rustc-dev-guide]: https://rustc-dev-guide.rust-lang.org/building/how-to-build-and-run.html#building-the-compiler
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm... this seems ok-ish to mention, but it's odd having that in the regular error message docs. That said... let's leave it in, it's pretty obvious to any reader whether it's relevant for them

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's here partly because using ˋxpy build rustdocˋ does not build the standard library for the same stage so ˋrustup toolchain link ...ˋ works but calling the relevant compiler or rustdoc fails with this error. It's either put it here or in the dev guide but this has a lot more visibility and it doesn't take much space. This particular code shouldn't come up too often for "regular" Rust usage too and so should not confuse many people

@oli-obk
Copy link
Contributor

oli-obk commented Feb 3, 2021

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Feb 3, 2021

📌 Commit dd6617ee35c3a81d3f0a20ab2bd7f1de1dcddd30 has been approved by oli-obk

@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 Feb 3, 2021
@rust-log-analyzer

This comment has been minimized.

@jyn514
Copy link
Member Author

jyn514 commented Feb 3, 2021

Ugh, I hate submodules.

This comes up a lot when bootstrapping.
@jyn514
Copy link
Member Author

jyn514 commented Feb 3, 2021

@bors r=oli-obk

@bors
Copy link
Contributor

bors commented Feb 3, 2021

📌 Commit 82914a5 has been approved by oli-obk

Dylan-DPC-zz pushed a commit to Dylan-DPC-zz/rust that referenced this pull request Feb 3, 2021
Add more information to the error code for 'crate not found'

This comes up a lot when bootstrapping.
m-ou-se added a commit to m-ou-se/rust that referenced this pull request Feb 4, 2021
Add more information to the error code for 'crate not found'

This comes up a lot when bootstrapping.
m-ou-se added a commit to m-ou-se/rust that referenced this pull request Feb 4, 2021
Add more information to the error code for 'crate not found'

This comes up a lot when bootstrapping.
m-ou-se added a commit to m-ou-se/rust that referenced this pull request Feb 5, 2021
Add more information to the error code for 'crate not found'

This comes up a lot when bootstrapping.
bors added a commit to rust-lang-ci/rust that referenced this pull request Feb 5, 2021
Rollup of 15 pull requests

Successful merges:

 - rust-lang#79554 (Generic associated types in trait paths)
 - rust-lang#80726 (relax adt unsizing requirements)
 - rust-lang#81307 (Handle `Span`s for byte and raw strings and add more detail )
 - rust-lang#81318 (rustdoc-json: Fix has_body)
 - rust-lang#81456 (Make remote-test-server easier to use with new targets)
 - rust-lang#81497 (rustdoc: Move `display_fn` struct inside `display_fn`)
 - rust-lang#81500 (Remove struct_type from union output)
 - rust-lang#81542 (Expose correct symlink API on WASI)
 - rust-lang#81676 (Add more information to the error code for 'crate not found')
 - rust-lang#81682 (Add additional bitset benchmarks)
 - rust-lang#81730 (Make `Allocator` object-safe)
 - rust-lang#81763 (Cleanup rustdoc pass descriptions a bit)
 - rust-lang#81767 (Update LayoutError/LayoutErr stability attributes)
 - rust-lang#81771 (Indicate change in RSS from start to end of pass in time-passes output)
 - rust-lang#81781 (Fix `install-awscli.sh` error in CI)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 29371c2 into rust-lang:master Feb 5, 2021
@rustbot rustbot added this to the 1.51.0 milestone Feb 5, 2021
@jyn514 jyn514 deleted the crate-not-found branch February 5, 2021 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-error-codes Area: Explanation of an error code (--explain) 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.

7 participants