nix-community / nixvim

Configure Neovim with Nix! [maintainer=@GaetanLepage, @traxys, @mattsturgeon, @khaneliman]
https://nix-community.github.io/nixvim
MIT License
1.6k stars 249 forks source link

[BUG] `pylsp` not building #1387

Closed pinarruiz closed 5 months ago

pinarruiz commented 5 months ago
Field Description
Plugin pylsp
Nixpkgs unstable (ff0dbd94265ac470dda06a657d5fe49de93b4599)
Home Manager unstable (c0ef0dab55611c676ad7539bf4e41b3ec6fa87d2)

Description

pylsp failing to build, i posted #1378 and just updated the lock, but now it seems like pylsp wont build.

Error log:

error:
       … while calling the 'head' builtin

         at /nix/store/8mk40j3mfrvndrnh89azngcgdgr1fidi-source/lib/attrsets.nix:922:11:

          921|         || pred here (elemAt values 1) (head values) then
          922|           head values
             |           ^
          923|         else

       … while evaluating the attribute 'value'

         at /nix/store/8mk40j3mfrvndrnh89azngcgdgr1fidi-source/lib/modules.nix:807:9:

          806|     in warnDeprecation opt //
          807|       { value = builtins.addErrorContext "while evaluating the option `${showOption loc}':" value;
             |         ^
          808|         inherit (res.defsFinal') highestPrio;

       (stack trace truncated; use '--show-trace' to show the full trace)

       error: attribute 'propagatedBuildInputs' missing

       at /nix/store/7qlh1mw3xl9iwizc0s3rzzjpwcj5b1gm-source/plugins/lsp/language-servers/pylsp.nix:626:55:

          625|             old: {
          626|               propagatedBuildInputs = pylspPlugins ++ old.propagatedBuildInputs;
             |                                                       ^
          627|               disabledTests =

Minimal, Reproducible Example (MRE)

lsp = {
        enable = true;
        servers = {
          ansiblels.enable = true;
          bashls.enable = true;
          clangd.enable = true;
          cmake.enable = true;
          dockerls.enable = true;
          emmet_ls.enable = true;
          eslint.enable = true;
          helm-ls.enable = true;
          html.enable = true;
          jsonls.enable = true;
          ltex.enable = true;
          lua-ls.enable = true;
          marksman.enable = true;
          nil_ls.enable = true;
          nixd.enable = true;
          pylsp.enable = true;
          # pylyzer.enable = true;
          pyright.enable = true;
          tailwindcss.enable = true;
          taplo.enable = true;
          terraformls.enable = true;
          texlab.enable = true;
          tsserver.enable = true;
          yamlls.enable = true;
        };
      };
    };

Edit: Fix package name

PerchunPak commented 5 months ago

Known issue https://github.com/NixOS/nixpkgs/issues/295735

Edit: Was confused by name, this issue is about pylsp, not pylyzer.

I bisected nixpkgs, and this commit is responsible for this issue: https://github.com/NixOS/nixpkgs/commit/a5028a61cfe408885780214c446274304eae430c

pinarruiz commented 5 months ago

Known issue NixOS/nixpkgs#295735

Edit: Was confused by name, this issue is about pylsp, not pylyzer.

I bisected nixpkgs, and this commit is responsible for this issue: NixOS/nixpkgs@a5028a6

Yeah I got it wrong sorry, just got edited :smile:

pinarruiz commented 5 months ago

Works thanks to #1369