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

lsp: disable warning notifications on server health warning by default #518

Closed
mrcjkb opened this issue Sep 23, 2024 · 1 comment · Fixed by #519
Closed

lsp: disable warning notifications on server health warning by default #518

mrcjkb opened this issue Sep 23, 2024 · 1 comment · Fixed by #519

Comments

@mrcjkb
Copy link
Owner

mrcjkb commented Sep 23, 2024

@edevil since this PR, I sometimes get spammed with warnings like:

cargo check failed to start: Cargo watcher failed, the command produced no valid metadata (exit code: ExitStatus(unix_wait_status(25856))):
    Blocking waiting for file lock on package cache
    Updating crates.io index
error: failed to write /nix/store/5dh2vfghjizfwgci0z00hbrpx5vwv6zc-rust-lib-src/std/Cargo.lock

Caused by:
  failed to open: /nix/store/5dh2vfghjizfwgci0z00hbrpx5vwv6zc-rust-lib-src/std/Cargo.lock

Caused by:
  Read-only file system (os error 30)

Failed to read Cargo metadata with dependencies for `/nix/store/5dh2vfghjizfwgci0z00hbrpx5vwv6zc-rust-lib-src/std/Cargo.toml`: Failed to run `cd "/
nix/store/5dh2vfghjizfwgci0z00hbrpx5vwv6zc-rust-lib-src/std" && RUSTUP_TOOLCHAIN="/nix/store/p024jn1sd4pn33lg0kaqxh7j2sp1iw7x-rustc-1.80.1" "/nix/s
tore/c0r47wb8xm1dbsvppw6gh75jfjakal1y-cargo-1.80.1/bin/cargo" "metadata" "--format-version" "1" "--all-features" "--manifest-path" "/nix/store/5dh2
vfghjizfwgci0z00hbrpx5vwv6zc-rust-lib-src/std/Cargo.toml" "--filter-platform" "x86_64-unknown-linux-gnu"`: `cargo metadata` exited with an error:
   Updating crates.io index
error: failed to write /nix/store/5dh2vfghjizfwgci0z00hbrpx5vwv6zc-rust-lib-src/std/Cargo.lock

Caused by:
  failed to open: /nix/store/5dh2vfghjizfwgci0z00hbrpx5vwv6zc-rust-lib-src/std/Cargo.lock

Caused by:
  Read-only file system (os error 30)
Run ':RustLsp logFile' for details.
Press ENTER or type command to continue

I think we might need to disable warnings when the server health level is warning by default,
with an config to enable them.
This is far too noisy.

Originally posted by @mrcjkb in #509 (comment)

@edevil
Copy link
Contributor

edevil commented Sep 23, 2024

Yeah, that's probably Ok. TBH only the errors seem interesting so far.

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 a pull request may close this issue.

2 participants