Repository navigation
x.py test specific directory requires platform specific path separator #79334
Description
On Windows, I tried this code:
python x.py test src/test/ui/expr/compound-assignment/eval-order.rs --pass runI expected to execute the file and give back a test result. Instead, it found zero results.
After an hour of flailing, I tired:
python x.py test src/test/ui\expr\compound-assignment\eval-order.rs --pass runAnd to my (pleasant) surprised, it found the file.
Ideally, it shouldn't matter which path separator is used at any point.
Metadata
Metadata
Assignees
Labels
Area: Makes things more difficult for new or seasoned contributors to RustArea: The testsuite used to check the correctness of rustcCategory: This is a bug.Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.Call for participation: This issue has a mentor. Use #t-compiler/help on Zulip for discussion.Operating system: WindowsRelevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)
Activity
jyn514 commentedon Aug 8, 2022
This is an issue with libtest - compiletest passes in
expr/compound-assignment/eval-order.rsas the filter, which libtest counts as filtering outexpr\compound-assignment\eval-order.rstoo.I think the workaround is to manually turn forward slashes into backslashes in
impl Step for test::Compiletestwhen running on windows.czzrr commentedon Aug 9, 2022
@rustbot claim
Rollup merge of rust-lang#100811 - czzrr:master, r=Mark-Simulacrum
Rollup merge of rust-lang#100811 - czzrr:master, r=Mark-Simulacrum
Rollup merge of rust-lang#100811 - czzrr:master, r=Mark-Simulacrum
Rollup merge of rust-lang#100811 - czzrr:master, r=Mark-Simulacrum
Rollup merge of rust-lang#100811 - czzrr:master, r=Mark-Simulacrum
jyn514 commentedon Sep 19, 2022
Looks like this was fixed in #100811 :) thanks @czzrr !