Skip to content

Expand rustup show - #406

Merged
brson merged 3 commits into
rust-lang:masterfrom
brson:show
May 7, 2016
Merged

brson merged 3 commits into
rust-lang:masterfrom
brson:show

Conversation

@brson

@brson brson commented May 7, 2016

Copy link
Copy Markdown
Contributor

Now it shows the active toolchain, the installed targets for the
active toolchain, and the installed toolchains. When there are
no additional toolchains or targets, the category headers
for them are not displayed.

@brson

brson commented May 7, 2016 •

Copy link
Copy Markdown
Contributor Author

Here's what it looks like with all categories of info displayed, since the test cases aren't very readable:

brian@ip-10-234-25-118:~/dev/rustup.rs⟫ rustup show
installed toolchains
--------------------

stable-x86_64-unknown-linux-gnu
nightly-i686-unknown-linux-gnu
nightly-x86_64-unknown-linux-gnu (default)

installed targets for active toolchain
--------------------------------------

i686-unknown-linux-gnu
x86_64-unknown-linux-gnu

active toolchain
----------------

nightly-x86_64-unknown-linux-gnu (default)
rustc 1.10.0-nightly (8da2bcac5 2016-04-28)

And the headers are bold. But it might be better to make everything but the headers bold.

@brson

brson commented May 7, 2016

Copy link
Copy Markdown
Contributor Author

@alexcrichton

Copy link
Copy Markdown
Member

r=me, nice!

I kinda like the bold headers, but the bold toolchains looks a little too much

brson added 3 commits May 7, 2016 17:37
Now it shows the active toolchain, the installed targets for the
active toolchain, and the installed toolchains. When there are
no additional toolchains or targets, the category headers
for them are not displayed.
Particulars of the MULTI_ARCH1 mock target mean these
tests can't run on i686-unknown-linux-gnu.
@brson
brson merged commit 45c0489 into rust-lang:master May 7, 2016
Sign up for free to 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.

2 participants