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

Dump all accessed environment variables when rustc fails #64574

Closed
wants to merge 3 commits into from

Conversation

gnzlbg
Copy link
Contributor

@gnzlbg gnzlbg commented Sep 18, 2019

When reading environment variables in boostrap/bin/rustc, this PR caches the read environment variable, and prints them on failure, to allow users to c&p them before the rustc invocation when trying to reproduce.

r? @oli-obk

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 18, 2019
@gnzlbg
Copy link
Contributor Author

gnzlbg commented Sep 18, 2019

cc @alexcrichton

@petrochenkov
Copy link
Contributor

cc #64316
(Looks like this won't work once that PR lands.)

@pnkfelix
Copy link
Member

(cc #38686, an issue I filed regarding this exact problem ages ago...)

@gnzlbg
Copy link
Contributor Author

gnzlbg commented Sep 18, 2019

This isn't necessary. --on-fail=bash spawns an interactive shell with the environment already configured such that the last command can be re-run.

@gnzlbg gnzlbg closed this Sep 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants