Skip to content

Fix compile-fail tests on windows #27974

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

Merged
merged 1 commit into from
Aug 25, 2015
Merged

Conversation

Diggsey
Copy link
Contributor

@Diggsey Diggsey commented Aug 24, 2015

Fixes #27952

@rust-highfive
Copy link
Contributor

r? @nikomatsakis

(rust_highfive has picked a reviewer for you, use r? to override)

@alexcrichton
Copy link
Member

Is there a way the error in #27952 can be reproduced locally? Our bots have been running cfail tests for quite some time, so I'm curious as to what happened to make everything go wrong

@Diggsey
Copy link
Contributor Author

Diggsey commented Aug 24, 2015

The bug is that the cfail tests pass when they shouldn't on windows: the bots don't detect it because it's too meta: there's nothing to check that the cfail tests will actually fail!

Reproducing is easy: just introduce an additional compile error to one of the cfail tests, so that now rustc outputs both errors. On linux this would cause the test to fail, because an unexpected error was detected, but on windows the additional error is ignored because it doesn't match the pattern that it thinks all errors follow.

@alexcrichton
Copy link
Member

@bors: r+ 19dc4d0

Ah ok, thanks!

@bors
Copy link
Collaborator

bors commented Aug 25, 2015

⌛ Testing commit 19dc4d0 with merge 28ef44a...

bors added a commit that referenced this pull request Aug 25, 2015
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.

5 participants