Remove the pre-defined `environment.systemPackages` in `infra/common/default.nix` #25

New issue

Closed

opened 2024-11-21 12:11:33 +01:00 by Niols · 3 comments

Niols commented

2024-11-21 12:11:33 +01:00

Owner

As of today, infra/common/default.nix includes the following lines:

   environment.systemPackages = with pkgs; [
     (pkgs.vim_configurable.customize {
       name = "vim";
       vimrcConfig.packages.myplugins = with pkgs.vimPlugins; {
         start = [ vim-nix ]; # load plugin on startup
       };
       vimrcConfig.customRC = ''
         " your custom vimrc
         set nocompatible
         set backspace=indent,eol,start
         " Turn on syntax highlighting by default
         syntax on
         " ...
       '';
     })
     wget
     subversion
   ];

I'm not happy with them and neither is @fricklerhandwerk. I would be up for removing them altogether. The machines are Nix-capable, so if this is for very rare use, one can just nix run nixpkgs#vim or something like that, anyway.

As of today, `infra/common/default.nix` includes the following lines: https://git.fediversity.eu/Fediversity/Fediversity/src/commit/d2638845d0f0edf78c20f2aef300cb05daa7a93d/infra/common/default.nix#L19-L36 I'm not happy with them and [neither is @fricklerhandwerk](https://git.fediversity.eu/Fediversity/Fediversity/pulls/22#issuecomment-3269). I would be up for removing them altogether. The machines are Nix-capable, so if this is for very rare use, one can just `nix run nixpkgs#vim` or something like that, anyway.

Niols added the

question

label 2024-11-21 12:11:33 +01:00

Niols referenced this issue

2024-11-21 12:12:06 +01:00

Factorise other configuration options #22

Niols commented

2025-02-21 19:35:14 +01:00

Author

Owner

@kiara > WDYT about this? Important for Procolix devs, or can we remove safely?

kiara commented

2025-02-21 19:53:05 +01:00

Owner

@Niols: feel free to remove.

such config was used on git-less nixos machines by @kevin, but that was only under the presumption config changes would need to be done over ssh, rather than from repos checked out locally to then deploy.
in other words this seemed copied from the historical situation, rather than thought out with our present setup in mind.

@Niols: feel free to remove. such config was used on git-less nixos machines by @kevin, but that was only under the presumption config changes would need to be done over ssh, rather than from repos checked out locally to then deploy. in other words this seemed copied from the historical situation, rather than thought out with our present setup in mind.

Niols referenced this issue from a pull request that will close it,

2025-02-21 20:29:31 +01:00

Remove environment.systemPackages from VMs #176

Niols commented

2025-02-21 20:29:41 +01:00

Author

Owner

Awesome; thank you very much for the details :-)