neovim: move LUA_(C)PATH setup from wrapping to init.lua#8606
Merged
teto merged 1 commit intonix-community:masterfrom Jan 22, 2026
Merged
neovim: move LUA_(C)PATH setup from wrapping to init.lua#8606teto merged 1 commit intonix-community:masterfrom
teto merged 1 commit intonix-community:masterfrom
Conversation
so alternative GUIs can find lua dependencies that are not plugins (for instance nio). Some plugins depend on lua packages that are not vim plugins and thus become "invisible" to nixpkgs as it stands. For now they need to be specified manually via `extraLuaPackages` but hopefully we can autodiscover those in nixpkgs.
teto
force-pushed
the
teto/neovim-inform-lua-dependencies
branch
from
38fbbcd to
239fc6a
Compare
khaneliman
approved these changes
Jan 21, 2026
Collaborator
Author
|
I was waiting to see if previous PR raised any complaint. Usually it arrives 30mn after a merge. It has been 3 days so I assume it's fine. With this one can finally run neovide (or any neovim derivative) with the generated config. No more wrapping needed. |
6 tasks
BerkeleyTrue
added a commit
to BerkeleyTrue/dotfiles
that referenced
this pull request
Jan 23, 2026
nix-community/home-manager#8606 breaks prev behavior and tries to clobber users init.lua
9 tasks
Collaborator
Author
|
seems like I forgot to remove |
teto
added a commit
to teto/home-manager
that referenced
this pull request
Jan 25, 2026
turns out I forgot to remove the now unnecessary wrapper arguments in nix-community#8606. This meant that `xdg.configFile."nvim/init.lua".enable = false` was enough to fix the user issues but this should not be the case anymore.
teto
added a commit
to teto/home-manager
that referenced
this pull request
Jan 29, 2026
turns out I forgot to remove the now unnecessary wrapper arguments in nix-community#8606. This meant that `xdg.configFile."nvim/init.lua".enable = false` was enough to fix the user issues but this should not be the case anymore.
teto
added a commit
that referenced
this pull request
Jan 29, 2026
turns out I forgot to remove the now unnecessary wrapper arguments in #8606. This meant that `xdg.configFile."nvim/init.lua".enable = false` was enough to fix the user issues but this should not be the case anymore.
repparw
pushed a commit
to repparw/home-manager
that referenced
this pull request
Jan 30, 2026
turns out I forgot to remove the now unnecessary wrapper arguments in nix-community#8606. This meant that `xdg.configFile."nvim/init.lua".enable = false` was enough to fix the user issues but this should not be the case anymore.
kugland
pushed a commit
to kugland/home-manager
that referenced
this pull request
Feb 13, 2026
turns out I forgot to remove the now unnecessary wrapper arguments in nix-community#8606. This meant that `xdg.configFile."nvim/init.lua".enable = false` was enough to fix the user issues but this should not be the case anymore.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation is similar to #8586: not relying on wrapped arguments make the neovim install more "natural", or what users are used to. Alternative GUIs can find lua dependencies that are not plugins (for instance nio) without wrapping.
Some plugins depend on lua packages that are not vim plugins and thus become "invisible" to nixpkgs as it stands.
For now they need to be specified manually via
extraLuaPackagesbut hopefully we can autodiscover those in nixpkgs.I will wait for a bit before merging, to see if #8586 triggered any issue.
TODO:
enrich the path only if extraLuaPackages is setDescription
Checklist
Change is backwards compatible.
Code formatted with
nix fmtornix-shell -p treefmt nixfmt deadnix keep-sorted nixf-diagnose --run treefmt.Code tested through
nix run .#tests -- test-allornix-shell --pure tests -A run.all.Test cases updated/added. See example.
Commit messages are formatted like
See CONTRIBUTING for more information and recent commit messages for examples.
If this PR adds a new module
If this PR adds an exciting new feature or contains a breaking change.