Skip to content

rustc: Remove def::DefUse#30354

Merged
alexcrichton merged 1 commit intorust-lang:masterfrom
petrochenkov:defuse
Dec 17, 2015
Merged

rustc: Remove def::DefUse#30354
alexcrichton merged 1 commit intorust-lang:masterfrom
petrochenkov:defuse

Conversation

@petrochenkov
Copy link
Contributor

@petrochenkov petrochenkov commented Dec 12, 2015

A relic of some old resolution algorithm?

@rust-highfive
Copy link
Contributor

rust-highfive commented Dec 12, 2015

r? @nikomatsakis

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

@sanxiyn
Copy link
Contributor

sanxiyn commented Dec 14, 2015

@bors r+

@bors
Copy link
Collaborator

bors commented Dec 14, 2015

📌 Commit 54b1815 has been approved by sanxiyn

@bors
Copy link
Collaborator

bors commented Dec 14, 2015

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

@bors
Copy link
Collaborator

bors commented Dec 14, 2015

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

@petrochenkov
Copy link
Contributor Author

petrochenkov commented Dec 14, 2015

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

Needs retry

@sanxiyn
Copy link
Contributor

sanxiyn commented Dec 14, 2015

@bors retry

@bors
Copy link
Collaborator

bors commented Dec 15, 2015

⌛ Testing commit 54b1815 with merge cb2b9d0...

@bors
Copy link
Collaborator

bors commented Dec 15, 2015

💔 Test failed - auto-mac-32-opt

@Manishearth
Copy link
Member

Manishearth commented Dec 15, 2015

@bors retry

Failure caused by me shutting down the broken mac5 builder.

@sanxiyn
Copy link
Contributor

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
Collaborator

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

petrochenkov commented Dec 16, 2015

Rebased.

@sanxiyn
Copy link
Contributor

sanxiyn commented Dec 16, 2015

@bors r+

@bors
Copy link
Collaborator

bors commented Dec 16, 2015

📌 Commit 0cc69f0 has been approved by sanxiyn

@bors
Copy link
Collaborator

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

alexcrichton commented Dec 17, 2015

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