Skip to content

Add test for #24389 #28707

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
Sep 29, 2015
Merged

Add test for #24389 #28707

merged 1 commit into from
Sep 29, 2015

Conversation

apasel422
Copy link
Contributor

Closes #24389.

@rust-highfive
Copy link
Contributor

r? @nikomatsakis

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

@alexcrichton
Copy link
Member

@bors: r+ 47c176e

Thanks!

@bors
Copy link
Collaborator

bors commented Sep 28, 2015

⌛ Testing commit 47c176e with merge 3002a1b...

@bors
Copy link
Collaborator

bors commented Sep 28, 2015

💔 Test failed - auto-linux-64-opt

@apasel422
Copy link
Contributor Author

The test failure is unrelated to this PR.

@apasel422
Copy link
Contributor Author

Why did auto-mac-32-opt succeed with this but auto-linux-64-opt did not?

@apasel422
Copy link
Contributor Author

It's unclear why this test failed in the first place (it's not related to the warning on stderr), so it seems to be spurious.

@alexcrichton
Copy link
Member

@bors: retry

(signal 11 spurious failure)

bors added a commit that referenced this pull request Sep 29, 2015
@bors
Copy link
Collaborator

bors commented Sep 29, 2015

⌛ Testing commit 47c176e with merge abfbea2...

@bors bors merged commit 47c176e into rust-lang:master Sep 29, 2015
@Diggsey
Copy link
Contributor

Diggsey commented Sep 29, 2015

👍

@apasel422 apasel422 deleted the issue-24389 branch September 29, 2015 15:38
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.

6 participants