overlays/csharp-ls: Dont use dotnet from env

LSP has a specific dotnet version it needs... use the latest dotnet 8 pkg for it to avoid workspace conflicts.
khaneliman · May 17, 2024 · 1c9fc5f · 1c9fc5f
1 parent 5529cb3
commit 1c9fc5f
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/modules/home/programs/terminal/editors/neovim/default.nix b/modules/home/programs/terminal/editors/neovim/default.nix
@@ -16,8 +16,6 @@ in
   config = mkIf cfg.enable {
     home = {
       sessionVariables = {
-        # TODO: why was this set ?
-        # DOTNET_ROOT = "${pkgs.dotnet-sdk_8}";
         EDITOR = mkIf cfg.default "nvim";
       };
     };

diff --git a/overlays/csharp-ls/default.nix b/overlays/csharp-ls/default.nix
@@ -0,0 +1,8 @@
+_: _final: prev: {
+  csharp-ls = prev.csharp-ls.overrideAttrs (_oldAttrs: {
+    # NOTE: csharp-ls requires a very new dotnet 8 sdk. This causes issues with workspace dotnet
+    # collisions because dotnet commands will run off the newest SDK breaking working with lower
+    # version projects.
+    useDotnetFromEnv = false;
+  });
+}