Skip to content

add/remove multiple toolchains - #986

Merged
bors merged 2 commits into
rust-lang:masterfrom
durka:add-rm-multiple
Mar 18, 2017
Merged

add/remove multiple toolchains#986
bors merged 2 commits into
rust-lang:masterfrom
durka:add-rm-multiple

Conversation

@durka

@durkadurka commented Mar 14, 2017

Copy link
Copy Markdown
Contributor

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

Depends on #968.
Fixes#976.

@brson

Copy link
Copy Markdown
Contributor

Thanks @durka can you add test cases? I think there are test cases to crib off of in cli-v2.rs, and the tests can go in that file as well.

@durka

durka commented Mar 17, 2017

Copy link
Copy Markdown
ContributorAuthor

Test added. It passes locally on OSX, but wait for CI.

@durka
durkaforce-pushed the add-rm-multiple branch 5 times, most recently from 967ca55 to 3aaf0acCompareMarch 17, 2017 20:52
@brson

Copy link
Copy Markdown
Contributor

@bors r+

@bors

bors commented Mar 17, 2017

Copy link
Copy Markdown
Contributor

📌 Commit a807244 has been approved by brson

@bors

bors commented Mar 18, 2017

Copy link
Copy Markdown
Contributor

⌛ Testing commit a807244 with merge 02d1b87...

bors added a commit that referenced this pull request Mar 18, 2017
add/remove multiple toolchains
Teaches the `toolchain` subcommand to accept multiple arguments for toolchains to add or remove.
Depends on #968.
Fixes#976.
@bors

bors commented Mar 18, 2017

Copy link
Copy Markdown
Contributor

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

@bors
bors merged commit a807244 into rust-lang:masterMar 18, 2017
bors added a commit that referenced this pull request Apr 1, 2017
Add/remove multiple components+targets
Same as #986 but for components and targets. After this I don't see any more places to add `.multiple(true)`.
cc #1005
bors added a commit to rust-lang/cargo that referenced this pull request Jul 28, 2017
cargo install multiple crates
rust-lang/rustup#986 for `cargo install`
Revives #2601@pwoolcoc, replaces #3075@esclear, closes#2585 @kindlychung @cyplo
Avoids the sticking point of the previous two PRs (multiple registry updates) by threading through a first-run boolean flag to decide whether `select_pkg` needs to call `source.update()`.
There is still the issue that flags such as `--git` and `--vers` are "global" to the multiple packages you may be installing. The workaround is just to run `cargo install` separately. In the future we could add syntax like `cargo install foo=1.0 bar=2.5 quux=git://github.com/durka/quux#dev-branch` or something.
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.

3 participants

@durka@brson@bors