search

nix-community / plasma-manager

Manage KDE Plasma with Home Manager
https://nix-community.github.io/plasma-manager/
MIT License
600 stars 70 forks source link

rust-analyzer as an LSP in kate stopped working #197

Open Asqiir opened 3 months ago

Asqiir commented 3 months ago

This regression was introduced in #191 which updated kate from 23.08 to 24.05. I believe it was this specific commit from the kate project.

I could fix this locally with adding the old version of rust-analyzer as a custom setting.

  1. Can somebody on a different setup check whether they can reproduce this?
  2. If so, can we figure out (with reasonable effort), what the root cause is?
  3. If not, should we add my hacky patch to plasma-manager's kate?
Asqiir commented 3 months ago

and Somebody On Reddit has the same issue

ugur-a commented 2 months ago

hi, it seems like you forgot to close this issue

Asqiir commented 2 months ago

@ugur-a I was hoping for somebody else to confirm (or reject) whether they have the same issue: If it's just me, we can close it. If it's a common issue, I would like to know to figure out a good fix.

ugur-a commented 2 months ago

Oh god sorry I totally misread this. So:

  1. I created a small config (literally just the rust-analyzer setup), and it does indeed fail to find the workspace when I don't have "rootIndicationFileNames" = ["Cargo.lock" "Cargo.toml"]; set.
  2. Unfortunately, I have no idea what the root cause could be. But the Reddit post's OP didn't say anything about plasma-manager (or home-manager in general), so nothing would indicate that it's the one to blame?
  3. Whatever that would be, I'd say your workaround couldn't hurt?
Asqiir commented 2 months ago

The root cause is that they removed these rootIndicatorFilenames from upstream kate. And if this is a common issue, somebody (I) should probably file a bug report.