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

compiletest overwrites PYTHONPATH #108779

Closed
albertlarsan68 opened this issue Mar 5, 2023 · 0 comments · Fixed by #108793
Closed

compiletest overwrites PYTHONPATH #108779

albertlarsan68 opened this issue Mar 5, 2023 · 0 comments · Fixed by #108793
Labels
A-testsuite Area: The testsuite used to check the correctness of rustc C-bug Category: This is a bug. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)

Comments

@albertlarsan68
Copy link
Member

I tried running x t tests/debuginfo/basic-types-globals-metadata.rs, with the PYTHONPATH env variable set, as a workaround for rust-lang/simpleinfra#254.
However, it does not work, because compiletest or bootstrap overwrites the env variable instead of simply appending/prepending.

@albertlarsan68 albertlarsan68 added A-testsuite Area: The testsuite used to check the correctness of rustc T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) C-bug Category: This is a bug. labels Mar 5, 2023
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this issue Mar 7, 2023
…r=compiler-errors

Append to existing `PYTHONPATH` in compiletest instead of overwriting

Fixes rust-lang#108779
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-testsuite Area: The testsuite used to check the correctness of rustc C-bug Category: This is a bug. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant