Skip to content

Ask to run rustup install on stderr#490

Merged
taiki-e merged 2 commits intotaiki-e:mainfrom
narpfel:ask_to_run-stderr
May 9, 2026
Merged

Ask to run rustup install on stderr#490
taiki-e merged 2 commits intotaiki-e:mainfrom
narpfel:ask_to_run-stderr

Conversation

@narpfel
Copy link
Copy Markdown
Contributor

@narpfel narpfel commented May 7, 2026

When stdout is not visible, e. g. when piping the output of cargo llvm-cov to a file, it seems like cargo llvm-cov hangs without doing anything when the llvm-tools-preview rustup component is not installed. stderr is less likely to be redirected, so asking on there is less likely to not be visible.

When `stdout` is not visible, e. g. when piping the output of
`cargo llvm-cov` to a file, it seems like `cargo llvm-cov` hangs without
doing anything when the `llvm-tools-preview` rustup component is not
installed. `stderr` is less likely to be redirected, so asking on there
is less likely to not be visible.
@narpfel
Copy link
Copy Markdown
Contributor Author

narpfel commented May 7, 2026

The test failures on nightly seem to be due to rust-lang/cargo#16712.

Copy link
Copy Markdown
Owner

@taiki-e taiki-e left a comment

Choose a reason for hiding this comment

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

Thanks!

@taiki-e taiki-e merged commit 4f50598 into taiki-e:main May 9, 2026
53 checks passed
@taiki-e
Copy link
Copy Markdown
Owner

taiki-e commented May 9, 2026

Published in 0.8.6.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants