Home-manager: Enabling neovim in options throws an error

[SenchoPens]

Description

Since this commit executing nixos-rebuild switch throws an error:

error: --- TypeError ---------------------------------------------------------------------------------------------------------------------------------------------------------------------- nix
at: (151:10) in file: /nix/store/d2h4pc8xx5b3nmsrqjyaxixlzsfqdkcv-source/pkgs/applications/editors/neovim/utils.nix

   150|       wrapperArgs = res.wrapperArgs
   151|       ++ [
      |          ^
   152|         "--add-flags" "-u ${writeText "init.vim" res.neovimRcContent}"

value is the partially applied built-in function 'trace' while a list was expected
(use '--show-trace' to show detailed location information)

Maintainer CC

@beardhatcode

Steps to reproduce

  home-manager.users.sencho.programs.neovim = {
    enable = true;
  };

}

Technical details

• system: 0
• host os: Linux 5.9.1, NixOS, 21.03.20201029.f155651 (Okapi)
• multi-user?: no
• sandbox: yes
• version: nix-env (Nix) 2.4pre20201029_bb8e837
• nixpkgs: /etc/nixpkgs

[berbiche]

@teto ping since git blame is pointing towards your changes to neovim/utils.nix on nixpkgs

[beardhatcode]

I cannot reproduce this issue.

But from what I see, @teto changed the nvim package in the nixpkgs to no longer use extraMakeWrapperArgs as a string. But to use wrapperArgs as an array. I do not have time to fix and test this now (I will tomorrow).

Edit: the patch below may fix it but i did not test it.
https://github.com/beardhatcode/home-manager/commit/5f33dfe82bc315b4043b64f26cd73a10c04aca65

[teto]

wow that was fast, are you using master ? the change doesn't seem to have reached unstable yet ! sorry that it broke your setup, I will try to fix it this evening.
With those changes, it's now possible for home-manager to write the neovim config directly in $XDG_CONFIG_HOME/nvim/init.vim, thus avoiding issues linked to -u. Here is how to do it (I currently write it to init.generated.vim)
https://github.com/teto/home-manager/blob/scratch/modules/programs/neovim.nix#L260

[SenchoPens]

Hi, I use flakes and yesterday tried to update to master. No problem, I have no particular need in the most recent home-manager, I will just use the older revision, but to you guys great thanks for working on the amazing home-manager! Hope I will be able to contribute something too in the future.

[colemickens]

This hit nixos-unstable-small at least, btw.

[reedrw]

The build no longer fails but programs.neovim.vimAlias seems to no longer work.

[beardhatcode]

This issue is fixed in nixpkgs (https://github.com/NixOS/nixpkgs/pull/102231) by @teto

[evalexpr]

@reedrw maybe we should create a new issue for that? I've noticed the same behavior

[teto]

no need to create another issue: I will have a look at it tomorrow.

[jonringer]

Should we wrap the application outside of nixpkgs?

There's nothing to guarantee alignment between home-manager and nixpkgs.

I think it would make sense for nixpkgs to provide a neovim-unwrapped packaged, but if someone were to be using home-manager, then we would probably want to wrap the settings within the context of home-manager.

[teto]

I think it would make sense for nixpkgs to provide a neovim-unwrapped packaged

It is already the case, it's even called neovim-unwrapped.

but if someone were to be using home-manager, then we would probably want to wrap the settings within the context of home-manager.

if home-manager generates $XDG_CONFIG_HOME/nvim/init.vim, then we shouldn't need to wrap neovim-unwrapped.

btw I've merged a fix for the vi(m)Alias issue https://github.com/NixOS/nixpkgs/pull/102605

[reedrw]

Seems like everything is good now. Thanks! 👍