Skip to content

Fix build #968

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
Merged
bors merged 1 commit into from
Mar 17, 2017
Merged

Fix build #968

bors merged 1 commit into from
Mar 17, 2017

Conversation

@nrc
Copy link
Member

@nrc nrc commented Feb 26, 2017

(on nightly)

Fix build
79b93f9 
(on nightly)
@KalitaAlexey KalitaAlexey mentioned this pull request Feb 28, 2017
Closed
@ishitatsuyuki ishitatsuyuki mentioned this pull request Mar 13, 2017
Closed
@durka durka mentioned this pull request Mar 14, 2017
Merged
@nodakai
Copy link
Contributor

nodakai commented Mar 15, 2017

I'm curious, how the stable compiler accepted the old version?

@brson
Copy link
Contributor

brson commented Mar 16, 2017

@bors r+

@bors
Copy link
Contributor

bors commented Mar 16, 2017

📌 Commit 79b93f9 has been approved by brson

@bors
Copy link
Contributor

bors commented Mar 17, 2017

⌛ Testing commit 79b93f9 with merge 417f28d...

bors added a commit that referenced this pull request Mar 17, 2017
@bors
Auto merge of #968 - nrc:build-fix, r=brson
417f28d 
Fix build

(on nightly)
@bors
Copy link
Contributor

bors commented Mar 17, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: brson
Pushing 417f28d to master...

@bors bors merged commit 79b93f9 into rust-lang:master Mar 17, 2017
@bors bors mentioned this pull request Mar 17, 2017
Closed
bors added a commit that referenced this pull request Mar 18, 2017
@bors
Auto merge of #986 - durka:add-rm-multiple, r=brson
02d1b87 
add/remove multiple toolchains

Teaches the `toolchain` subcommand to accept multiple arguments for toolchains to add or remove.

Depends on #968.
Fixes #976.
@Keruspe Keruspe mentioned this pull request Mar 24, 2017
Merged
@brson
Copy link
Contributor

brson commented Apr 5, 2017

@nodakai rustup is pinned to an old nightly presently and not tested on stable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@nrc @nodakai @brson @bors