-
Notifications
You must be signed in to change notification settings - Fork 13.1k
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
Print message that clarifies some components are allowed to fail during build. #49952
Comments
Miri failing to build is not an issue that breaks the build. See also #49933 which fixes miri |
As a matter of fact this happened also with 1.24.1: see last comment of #49052 . When will be there a 1.25.1 fixing this? |
I'm very confused. Why would it be relevant whether miri or clippy are built? They are not part of the distribution. I'm also not sure why you need a fix for a successful build (modulo miri). Can you elaborate? Are you doing distributions of rustc? If so, why do you need clippy and miri? They are nightly only. What commands are you invoking in order to get the above output? |
I do first I don't know why clippy or mire are relevant, but the above command fails, and I want to install on my system Rust. |
The failures are not failing your build. The final statement says it was successful, even if the two tools fail. You can use the result just like you can use one where all tools successfully built. The "normal" way to install Rust is not by building it yourself, but by using rustup. We could emit a message that says that these builds are allowed to fail and that you can ignore the failure. Do you think that would help? |
I think that such a clarifying message, stating that the immediately preceding failures shall be ignored, will be very useful. |
Triage: I've updated the title of the issue to more accurately reflect the current state of the issue. |
This should no longer be relevant now that toolstate no longer exists: #79249 |
Any ideas?
The text was updated successfully, but these errors were encountered: