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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃檹馃徎 REQ/enhancement: Add "Rust Analyzer: stop server" command #4697

Closed
rivy opened this issue Jun 1, 2020 · 8 comments 路 Fixed by #13426
Closed

馃檹馃徎 REQ/enhancement: Add "Rust Analyzer: stop server" command #4697

rivy opened this issue Jun 1, 2020 · 8 comments 路 Fixed by #13426
Labels
E-hard S-actionable Someone could pick this issue up and work on it right now

Comments

@rivy
Copy link

rivy commented Jun 1, 2020

The analyzer takes a lot of resources and that sometimes gets in the way of rapid iteration.
I would like to have the option to stop the service in those cases.

@ruabmbua
Copy link
Contributor

ruabmbua commented Jun 3, 2020

The vscode C++ plugin adds a button to the C/C++ menu in the bottom right to stop / restart analysis.

@matklad matklad added the E-hard label Jul 11, 2020
@lnicola lnicola added the S-actionable Someone could pick this issue up and work on it right now label Jan 17, 2021
@rivy rivy changed the title REQ/enhancement: Add "Rust Analyzer: stop server" command 馃檹馃徎 REQ/enhancement: Add "Rust Analyzer: stop server" command Jul 28, 2021
@rivy
Copy link
Author

rivy commented Jul 28, 2021

If this isn't going to be implemented, how about adding a user-customizable delay after file saves before re-compiling? That would allow faster CLI recompiles and code iteration.

@OWissett
Copy link

I am often finding myself writing code on the train (and subsequently having very slow internet). This is an issue with the rust analyzer, since it takes a slow time for it to fetch the metadata, and blocks me from doing other things, which is somewhat frustrating. I appreciate that this is probably not a very common problem (and I should probably just not write code on the train so much), however, this feature would be nice.

@lnicola
Copy link
Member

lnicola commented Sep 22, 2022

@OWissett you still need that metadata to compile your code. RA just does it a bit earlier.

@OWissett
Copy link

OWissett commented Sep 22, 2022

Is it required for the install of new crates? Since it would be convenient to be able to install new crates, so I can start writing code whilst the metadata is fetched. I admit I am not thoroughly acquainted with how the RA works.

Apologies if these are silly questions.

@lnicola
Copy link
Member

lnicola commented Sep 22, 2022

Yes, it's required in order to load the project. There's some discussion in https://rust-lang.zulipchat.com/#narrow/stream/185405-t-compiler.2Frust-analyzer/topic/how.20to.20fix.3A.20.20blocking.20waiting.20for.20file.20lock.20on.20build.20di.2E.2E.2E/near/299088097

The problem (for you) is that we use cargo metadata to determine the project structure. I think it will get better when HTTP/sparse registries are stabilized.

@OWissett
Copy link

Ah, thanks for the speedy replies.

@ditsuke
Copy link

ditsuke commented Oct 11, 2022

/dupe #12936

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
E-hard S-actionable Someone could pick this issue up and work on it right now
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants