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

Cross crate tests on windows are broken #5286

Closed
brson opened this issue Mar 8, 2013 · 6 comments
Closed

Cross crate tests on windows are broken #5286

brson opened this issue Mar 8, 2013 · 6 comments
Labels
A-testsuite Area: The testsuite used to check the correctness of rustc E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue. O-windows Operating system: Windows P-low Low priority

Comments

@brson
Copy link
Contributor

brson commented Mar 8, 2013

The compiletest driver is not setting up the aux paths correctly.

@yichoi
Copy link
Contributor

yichoi commented Mar 9, 2013

I've seen it before but I cannot reproduce it from incoming branch

@metajack
Copy link
Contributor

Have either of you seen this since?

@crnobog
Copy link

crnobog commented Jul 21, 2013

I've seen this today. For me the compiletest driver is building the executable as i686-pc-mingw32\test\run-pass\anon-extern-mod-cross-crate-2.stage2-i686-pc-mingw32.exe and the aux crate in i686-pc-mingw32\test\run-pass\anon-extern-mod-cross-crate-2.libaux

But then the PATH compiletest uses to run the executable has i686-pc-mingw32\test\run-pass\anon-extern-mod-cross-crate-2.stage2-i686-pc-mingw32.libaux in it, so it can't find the appropriate dlls.

I've had a look at runtest.rs but I haven't unravelled how these paths are constructed yet.

@crnobog
Copy link

crnobog commented Jul 21, 2013

Tracked it down, the issue is that the aux directory is determined during the build as output_base_name(config, testfile).with_filetype("libaux") which replaces the stage2 identifier with .libaux instead of appending it.
The win32 runner then just chops off the ".exe" from the test executable name to get the aux directory.

I think the safer way to fix it is to make the aux directory contain the stage identifier? I can't imagine you'd want different stages under test to share aux crates at any point.

@pnkfelix
Copy link
Member

This particular testing infrastructure issue is not a 1.0 blocker. Assigning P-low.

@brson brson mentioned this issue Aug 12, 2014
33 tasks
@brson
Copy link
Contributor Author

brson commented Jan 13, 2015

This looks to be fixed.

@brson brson closed this as completed Jan 13, 2015
bors added a commit to rust-lang-ci/rust that referenced this issue May 2, 2020
…p1995

check_pat: delay creation of the "normal" vec until we reach the branch where it is actually needed

changelog: none
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 E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue. O-windows Operating system: Windows P-low Low priority
Projects
None yet
Development

No branches or pull requests

5 participants