-
Notifications
You must be signed in to change notification settings - Fork 13.3k
test: Ignore some problematic tests on powerpc and powerpc64* #51392
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
Conversation
r? @aturon (rust_highfive has picked a reviewer for you, use r? to override) |
Note: I was unable to run the testsuite locally, the testsuite wouldn't get past the tidycheck script. I am therefore waiting for the CI to fail now so I can update the line numbers in the stderr files. CC @shepmaster |
The job Click to expand the log.
I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
@@ -13,6 +13,9 @@ | |||
// ignore-emscripten | |||
// ignore-mips | |||
// ignore-mips64 | |||
// ignore-powerpc |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this file is for x86, so this makes sense.
@@ -13,6 +13,9 @@ | |||
// ignore-aarch64 | |||
// ignore-asmjs | |||
// ignore-mips64 | |||
// ignore-powerpc |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't understand this. What platform is this not ignored for?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could someone review this before the 1.27.0 release? @shepmaster ? |
r? |
Can someone pick this up, please? Otherwise rustc 1.27 might not build on Debian/ppc64. Thanks. |
@bors r+ |
📌 Commit d63aabc has been approved by |
test: Ignore some problematic tests on powerpc and powerpc64* This updates the list of tests which can be safely ignored on powerpc and powerpc64*.
☀️ Test successful - status-appveyor, status-travis |
This updates the list of tests which can be safely ignored on powerpc and powerpc64*.