Add PATH in post-install message when not modifying PATH

[f-bro]

Fixes #374

Unix:

To get started you need Cargo's bin directory ($HOME/.cargo/bin) in your PATH
environment variable.

Windows:

To get started you need Cargo's bin directory (C:\Users\f-bro\.cargo\bin) in
your PATH environment variable. This has not been done automatically.

[Diggsey]

Thanks for the PR!

I think it would be good to unify some of the logic here, across windows and linux:

• Give canonical_cargo_home logic for both windows and non-windows.
• Switch to using canonical_cargo_home() on all platforms
• Add the bin path to the post_install_msg_win as well (it's currently missed)

[f-bro]

Yes, it would be more consistent.
I will have a look tomorrow.

[f-bro]

It looks like AppVeyor couldn't download rust-std:

Downloading rustup-init.exe (7,601,125 bytes)...100%
rustup-init.exe -y --default-toolchain=nightly-2017-04-25-x86_64-pc-windows-msvc
info: syncing channel updates for 'nightly-2017-04-25-x86_64-pc-windows-msvc'
info: downloading component 'rustc'
info: downloading component 'rust-std'

The console output stops here.

[Diggsey]

Probably just a network failure.

LGTM, except HOMEPATH should be USERPROFILE, (I just fixed that in the online editor)

@bors r+

[bors]

📌 Commit a2e465c has been approved by Diggsey

[bors]

⌛ Testing commit a2e465c with merge cc79a6e...

[bors]

☀️ Test successful - status-appveyor, status-travis
Approved by: Diggsey
Pushing cc79a6e to master...