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

Refine suggestions of sourcing $HOME/.cargo/env #3445

Merged
merged 1 commit into from
Aug 20, 2023

Conversation

rami3l
Copy link
Member

@rami3l rami3l commented Aug 16, 2023

Closes #3415.

Copy link
Contributor

@djc djc left a comment

Choose a reason for hiding this comment

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

Drive-by review: nice!

@rami3l
Copy link
Member Author

rami3l commented Aug 16, 2023

@rbtcollins The CI check failed spuriously and requires a rerun.

@rami3l rami3l force-pushed the fix/better-source-suggestions branch from e682122 to d861eba Compare August 16, 2023 12:57
@rami3l
Copy link
Member Author

rami3l commented Aug 18, 2023

Weird. After a rerun I'm still getting errors that should have nothing to do with my changes...

@djc
Copy link
Contributor

djc commented Aug 18, 2023

The armv7-linux issues are an upstream problem: https://rust-lang.zulipchat.com/#narrow/stream/242791-t-infra/topic/rustup.20CI.20fails.20to.20install.20Docker.20images.20for.20armv7-linux. macOS seems to be suffering from intermittent issues in different tests. I think we should just merge this despite the CI failures.

@rbtcollins rbtcollins merged commit a4dd7d0 into rust-lang:master Aug 20, 2023
15 of 16 checks passed
@rami3l rami3l deleted the fix/better-source-suggestions branch August 20, 2023 06:20
@rami3l rami3l added this to the 1.27.0 milestone Jan 18, 2024
@rami3l rami3l mentioned this pull request Jan 29, 2024
1 task
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.

rustup suggest executing "source ...", but this command is absent in dash
3 participants