New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
nicotine-plus: init at 1.4.1 #53717
nicotine-plus: init at 1.4.1 #53717
Conversation
I wrote an expression for this, but it's less polished than yours, so I'm recommending yours be merged instead. |
I think so too. However, it's up to core maintainers to decide (I couldn't find any note on their policy regarding pinned versions anywhere). |
pkgs/applications/networking/soulseek/nicotine-plus/default.nix
Outdated
Show resolved
Hide resolved
pkgs/applications/networking/soulseek/nicotine-plus/default.nix
Outdated
Show resolved
Hide resolved
pkgs/applications/networking/soulseek/nicotine-plus/default.nix
Outdated
Show resolved
Hide resolved
pkgs/applications/networking/soulseek/nicotine-plus/default.nix
Outdated
Show resolved
Hide resolved
pkgs/applications/networking/soulseek/nicotine-plus/default.nix
Outdated
Show resolved
Hide resolved
pkgs/applications/networking/soulseek/nicotine-plus/default.nix
Outdated
Show resolved
Hide resolved
pkgs/applications/networking/soulseek/nicotine-plus/default.nix
Outdated
Show resolved
Hide resolved
@GrahamcOfBorg build nicotine-plus |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Motivation for this change
https://github.com/Nicotine-Plus/nicotine-plus
Would be nice to have it.
Also, maybe it's a good idea to pin dependencies to specific versions?
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)