search

greshake / i3status-rust

Very resourcefriendly and feature-rich replacement for i3status, written in pure Rust
GNU General Public License v3.0
2.85k stars 470 forks source link

Add shell.nix for dev environment #1908

Closed cortices closed 7 months ago

cortices commented 1 year ago

Take 3. Should be just the commit for this feature. sorry about all the extra emails you must be getting!

MaxVerevkin commented 1 year ago

(Ignoring the CI failure) I'm a bit hesitant to merge nix-specific files into the project because I don't use this OS and thus can't guarantee that the file will be kept up-to-date. @ammgws @bim9262 anyone using nixos?

ammgws commented 1 year ago

I'm on Arch, but this feels out of place... Why does it need to be upstream?

bim9262 commented 1 year ago

I'm also on arch, but you're right it does feel out of place... but I don't know much about how nix works

bim9262 commented 1 year ago

Perhaps https://github.com/NixOS/nixpkgs would be more appropriate?

cortices commented 1 year ago

nixpkgs isn't really the right place, that's for packages (of which there already is one in home-manager instead). shell.nix file is to set up the requisite development dependencies. It shouldn't require changes -- it just tells nixOS what is needed to build a rust project, without necessarily having a rust env already on the machine. if you don't want it in the project, you don't have to keep it.

MaxVerevkin commented 1 year ago

Any thoughts on this being in the wiki instead?