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.

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

Expand `rustup show` #406

Merged
merged 3 commits into from May 7, 2016

Conversation

Projects
None yet
2 participants
@brson
Copy link
Contributor

brson commented May 7, 2016

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

This comment has been minimized.

Copy link
Contributor Author

brson commented May 7, 2016

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 force-pushed the brson:show branch from 1c238b7 to e9ea918 May 7, 2016

@brson

This comment has been minimized.

Copy link
Contributor Author

brson commented May 7, 2016

@alexcrichton

This comment has been minimized.

Copy link
Member

alexcrichton commented May 7, 2016

r=me, nice!

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

brson added some commits May 6, 2016

Expand `rustup show`
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.
Hack up some `show` tests to work everywhere
Particulars of the MULTI_ARCH1 mock target mean these
tests can't run on i686-unknown-linux-gnu.

@brson brson force-pushed the brson:show branch from e9ea918 to fafd0bb May 7, 2016

@brson brson merged commit 45c0489 into rust-lang:master May 7, 2016

1 of 2 checks passed

continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.