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

Explain manual installation in VS Code #81

Closed
wants to merge 1 commit into from

Conversation

cauthmann
Copy link

I spent way too much time on an offline installation, and I'm hoping this documentation change might save others the same trouble.

Copying rust-analyzer-{platform} into the path mentioned in the documentation (.../User/globalStorage/matklad.rust-analyzer) doesn't do anything - the globalState.serverVersion magic prevents the extension from using the file unless the extension has previously downloaded something there.

Manually providing a rust-analyzer binary always requires setting serverPath, and the documentation will now reflect that.

@lnicola
Copy link
Member

lnicola commented Dec 13, 2020

@matklad
Copy link
Member

matklad commented Dec 13, 2020

Curious, does cargo xtask install work for you? It should do all of the steps automatically.

@cauthmann
Copy link
Author

my bad: rust-lang/rust-analyzer#6858

@matklad: I haven't tested that. I don't need to install from source, I just wish to install the tools from local files, without requiring an internet connection.

bors bot added a commit to rust-lang/rust-analyzer that referenced this pull request Dec 13, 2020
6858: Docs: Explain manual installation in VS Code r=lnicola a=cauthmann

same as rust-analyzer/rust-analyzer.github.io#81 , just against the correct repository this time

Co-authored-by: Christian Authmann <8371862+cauthmann@users.noreply.github.com>
@matklad matklad closed this Dec 14, 2020
lnicola added a commit that referenced this pull request Jun 14, 2021
lnicola added a commit that referenced this pull request Jun 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants