Skip to content

Widen the range of entries seen as toolchains#620

Merged
Diggsey merged 2 commits intorust-lang:masterfrom
inejge:show-custom
Jul 28, 2016
Merged

Widen the range of entries seen as toolchains#620
Diggsey merged 2 commits intorust-lang:masterfrom
inejge:show-custom

Conversation

@inejge
Copy link
Copy Markdown
Contributor

@inejge inejge commented Jul 28, 2016

If only directory entries are chosen with is_dir(), the list will miss the symlinks/junctions to custom toolchains.

Fixes #603

If only directory entries are chosen with is_dir(), the list will
miss the symlinks/junctions to custom toolchains.

Fixes rust-lang#603
@Diggsey
Copy link
Copy Markdown
Contributor

Diggsey commented Jul 28, 2016

Thanks! Could you add a regression test for #603 ?

@inejge
Copy link
Copy Markdown
Contributor Author

inejge commented Jul 28, 2016

I've added a "show" output match to the existing test for rustup toolchain link.

@Diggsey Diggsey merged commit a71190c into rust-lang:master Jul 28, 2016
@inejge inejge deleted the show-custom branch July 29, 2016 18:33
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