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

Sort toolchains by semantic version. #688

Merged
merged 1 commit into from Aug 26, 2016

Conversation

Projects
None yet
3 participants
@malbarbo
Copy link
Contributor

malbarbo commented Aug 24, 2016

The current output of rustup toolchain list is sorted by the toolchain directory name. This pull request change the sort to be by semantic version.

This change the following output:

stable-x86_64-unknown-linux-gnu
beta-x86_64-unknown-linux-gnu
nightly-x86_64-unknown-linux-gnu
1.0.0-x86_64-unknown-linux-gnu
1.10.0-x86_64-unknown-linux-gnu
1.2.0-x86_64-unknown-linux-gnu

to:

stable-x86_64-unknown-linux-gnu
beta-x86_64-unknown-linux-gnu
nightly-x86_64-unknown-linux-gnu
1.0.0-x86_64-unknown-linux-gnu
1.2.0-x86_64-unknown-linux-gnu
1.10.0-x86_64-unknown-linux-gnu
@Boddlnagg

This comment has been minimized.

Copy link
Contributor

Boddlnagg commented Aug 24, 2016

This would fix #668 as far as I can see.

@brson brson merged commit c8cc167 into rust-lang:master Aug 26, 2016

1 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
@brson

This comment has been minimized.

Copy link
Contributor

brson commented Aug 26, 2016

Beautiful. Thanks @malbarbo !

@malbarbo malbarbo deleted the malbarbo:sort-semver branch Sep 30, 2016

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.