Skip to content

Fix several -Z orbit crater blockers. - #33488

Merged
bors merged 3 commits into
rust-lang:masterfrom
eddyb:trans-fixes
May 10, 2016
Merged

Fix several -Z orbit crater blockers.#33488
bors merged 3 commits into
rust-lang:masterfrom
eddyb:trans-fixes

Conversation

@eddyb

@eddybeddyb commented May 7, 2016

Copy link
Copy Markdown
Contributor

Fixes 3 of the issues found by @nikomatsakis' crater run with -Z orbit forced on:
https://gist.github.com/nikomatsakis/6688c30a0e5d3fed07cc1ebd4efb1412

Two of the regressions seemed to be fixed by #33130 and the remaining two are timeouts.

@rust-highfive

Copy link
Copy Markdown
Contributor

r? @pnkfelix

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

@eddybeddyb added the S-waiting-on-crater Status: Waiting on a crater run to be completed. label May 9, 2016
@eddyb

eddyb commented May 9, 2016

Copy link
Copy Markdown
ContributorAuthor

@nikomatsakis Could you try crater + -Z orbit again with this PR applied?

@nikomatsakis

Copy link
Copy Markdown
Contributor

@eddyb done.

@Aatch

Copy link
Copy Markdown
Contributor

@bors r+

@bors

bors commented May 10, 2016

Copy link
Copy Markdown
Collaborator

📌 Commit cb3a557 has been approved by Aatch

@eddybeddyb removed the S-waiting-on-crater Status: Waiting on a crater run to be completed. label May 10, 2016
@eddyb

Copy link
Copy Markdown
ContributorAuthor

Crater results, in case anyone cares. The only regression is just a (timeout) false positive.

@bors

bors commented May 10, 2016

Copy link
Copy Markdown
Collaborator

⌛ Testing commit cb3a557 with merge 04308ce...

bors added a commit that referenced this pull request May 10, 2016
Fix several -Z orbit crater blockers.
Fixes 3 of the issues found by @nikomatsakis' crater run with `-Z orbit` forced on:
https://gist.github.com/nikomatsakis/6688c30a0e5d3fed07cc1ebd4efb1412
Two of the regressions seemed to be fixed by #33130 and the remaining two are timeouts.
@bors
bors merged commit cb3a557 into rust-lang:masterMay 10, 2016
@eddyb
eddyb deleted the trans-fixes branch May 10, 2016 21:39
Sign up for freeto 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

@eddyb@rust-highfive@nikomatsakis@Aatch@bors@pnkfelix