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.

Sign up for GitHub

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

Jump to bottom

rustc: Remove def::DefUse #30354

Merged
merged 1 commit into from
Dec 17, 2015
Merged

rustc: Remove def::DefUse #30354

merged 1 commit into from
Dec 17, 2015

Conversation

petrochenkov
Copy link
Contributor

A relic of some old resolution algorithm?

@rust-highfive
Copy link
Collaborator

r? @nikomatsakis

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

@sanxiyn
Copy link
Member

sanxiyn commented Dec 14, 2015

@bors r+

@bors
Copy link
Contributor

bors commented Dec 14, 2015

📌 Commit 54b1815 has been approved by sanxiyn

@bors
Copy link
Contributor

bors commented Dec 14, 2015

⌛ Testing commit 54b1815 with merge 2471b51...

@bors
Copy link
Contributor

bors commented Dec 14, 2015

💔 Test failed - auto-mac-64-nopt-t

@petrochenkov
Copy link
Contributor Author

fatal: unable to write new index file
Stopping at 'src/compiler-rt'; script returned non-zero status.

Needs retry

@sanxiyn
Copy link
Member

sanxiyn commented Dec 14, 2015

@bors retry

@bors
Copy link
Contributor

bors commented Dec 15, 2015

⌛ Testing commit 54b1815 with merge cb2b9d0...

@bors
Copy link
Contributor

bors commented Dec 15, 2015

💔 Test failed - auto-mac-32-opt

@Manishearth
Copy link
Member

@bors retry

Failure caused by me shutting down the broken mac5 builder.

@sanxiyn
Copy link
Member

sanxiyn commented Dec 16, 2015

@bors retry

@Manishearth Manishearth mentioned this pull request Dec 16, 2015
Merged
bors added a commit that referenced this pull request Dec 16, 2015
@bors
Auto merge of #30410 - Manishearth:rollup, r=Manishearth
785a8a6 
- Successful merges: #30320, #30368, #30372, #30376, #30388, #30392
- Failed merges: #30354, #30389
@bors
Copy link
Contributor

bors commented Dec 16, 2015

☔ The latest upstream changes (presumably #30410) made this pull request unmergeable. Please resolve the merge conflicts.

@petrochenkov
Copy link
Contributor Author

Rebased.

@sanxiyn
Copy link
Member

sanxiyn commented Dec 16, 2015

@bors r+

@bors
Copy link
Contributor

bors commented Dec 16, 2015

📌 Commit 0cc69f0 has been approved by sanxiyn

@bors
Copy link
Contributor

bors commented Dec 17, 2015

⌛ Testing commit 0cc69f0 with merge 9687a8a...

bors added a commit that referenced this pull request Dec 17, 2015
@bors
Auto merge of #30354 - petrochenkov:defuse, r=sanxiyn
9687a8a 
A relic of some old resolution algorithm?
@alexcrichton
Copy link
Member

Everything passed here except the android builder which apparently disappeared, but this seems pretty unlikely to only fail on android, so I'm going to merge manually.

@alexcrichton alexcrichton merged commit 0cc69f0 into rust-lang:master Dec 17, 2015
@petrochenkov petrochenkov deleted the defuse branch September 21, 2016 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@nikomatsakis nikomatsakis

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@petrochenkov @rust-highfive @sanxiyn @bors @Manishearth @alexcrichton @nikomatsakis