-
Notifications
You must be signed in to change notification settings - Fork 9
tools: work around breaking change in rustup 1.28.0 #191
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
Conversation
tools/helios-build/src/main.rs
Outdated
.args(["show", "active-toolchain"]) | ||
.current_dir(pwd.as_ref()) | ||
.stdin(Stdio::null()) | ||
.stdout(Stdio::null()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we want to capture and log this instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In an ideal world where I'm not trying to perform an emergency fix to Omicron CI, maybe!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The output is not particularly useful, it just shows the current toolchain and where the override comes from, if any. The stderr is left in place, so that's logged to the console whenever an error occurs (including when the toolchain is not installed).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Totally makes sense
These things have a habit of sticking around so I did the logging part and pushed it onto the branch. Does that look alright? It produces log messages that look like:
|
Looks fine to me! Apparently I'm not allowed to approve my own PR and new changes require approval from someone other than the last pusher. |
githuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuub |
Co-authored-by: Joshua M. Clulow <[email protected]> Original-Commit: c278c40
This works around rust-lang/rustup#3985. Tested via Omicron's
cargo xtask releng
.