nixos/profiles/base: install vim w/nix-syntax plugin#201380
nixos/profiles/base: install vim w/nix-syntax plugin#201380Ma27 merged 1 commit intoNixOS:masterfrom
Conversation
|
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-ready-for-review/3032/1469 |
SuperSandro2000
left a comment
There was a problem hiding this comment.
this should help people a lot in creating less syntax errors
|
+1 on the intention but the growth in size is too much. vim and vim_configurable must have different compilation flags. Could you reduce it ? how much would it be with neovim (just curious) ? |
|
I still think the best thing to do is to upstream nix syntax highlighting to the vim tree. See here LnL7/vim-nix#47 I'm mostly waiting on some word from LnL7 if they would be alright letting someone take over maintenance of the syntax definitions. Or maybe the definitions could be added to the community repos or the official repos even as point of contact for vim maintainers. |
|
Opened vim/vim#11646 |
Considering that you most likely edit Nix code in the installer, that
seems like a useful thing.
The size of the ISO I got from
nix-build nixos/release.nix -A iso_minimal.x86_64-linux
is still at 877M.
2b1ed69 to
0b5a0cb
Compare
|
@ncfavier TIL, thanks! Indeed, this doesn't cause a significant increase of size, we're still at 877M now. I'd be in favor of merging it, we can still remove it again if vim actually includes Nix integration. |
Description of changes
Considering that you most likely edit Nix code in the installer, that seems like a useful thing.
The size of the ISO I got from
is still at 877M.
As suggested by @madonius.
Things done
sandbox = trueset innix.conf? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)nixos/doc/manual/md-to-db.shto update generated release notes