This repository has been archived by the owner on Dec 8, 2020. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 62
Rustup tight integration #243
Comments
Merged
@KalitaAlexey the issue for me was that I had the PATH added in my |
@wagenet, |
Fair enough :) I suspect setups like mine are not rare, so it might make sense to provide some troubleshooting information somewhere. The fix is easy, just not immediately obvious. |
@wagenet, |
This was referenced May 18, 2017
Merged
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Currently there are the following opened issues: #242, #238, #231, #225, #187.
I am going to reform an approach I have chosen.
At the first iteration, I am going to remove updating rustup since it is useless most time.
At the second iteration, I am going to reform how the extension checks if some path belongs to rustup.
Currently it checks if the result of
rustc --print=sysroot
contains".rustup"
. I made the wrong assumption because some people has it differently.It will use
rustup run nightly rustc --print=sysroot
.At this point those who don't use the nightly toolchain will show an error message saying that the nightly toolchain is not installed.
It will check with
rustup component list --toolchain nightly
.It will install with
rustup component install rls --toolchain nightly
.At the third iteration, I am going to make the extension install the nightly toolchain.
At this point users who have Rustup will no longer have any issues.
At the fourth iteration, I am going to make the extension install Rustup.At the fifth iteration, I am going to make the extension offer a user to choose which way one wants the extension to run.
Let me know if you want something to be done differently.
The text was updated successfully, but these errors were encountered: