search

bjesus / wttrbar

Custom module for showing the weather in Waybar, using the great wttr.in
MIT License
164 stars 26 forks source link

flake.nix: init #62

Closed khaneliman closed 3 months ago

khaneliman commented 3 months ago

Used for being able to build project from source on NixOS and allowing people to build from latest git commits on the source repository.

bjesus commented 3 months ago

Hey @khaneliman ! thank you for creating this. I have to admit I feel a bit uncomfortable with the idea of adding distro-specific stuff to the repository. IIUC this adds 3 new files, and doubles the length of the README, only for an experimental feature of Nix, while most Nix users can simply install the normal wttrbar NixPxg (which I just realized you're maintaining, so thank you very much for that!) and won't need this. IMHO distro-stuff should generally go in a distro-related repo, as distros are countless and otherwise this repo will just end up with tons of directories and files that I as a maintainer have on idea how to maintain.

Please let me know if you think I'm missing some important detail here. and again thank you for the help and for the official wttrbar NixPkg.

khaneliman commented 3 months ago

That’s fair enough, I understand. It’ll at least be in PR history if someone does need an example to implement locally for development.