-
-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
nixos/navidrome: fix settings type #312757
Conversation
1ce5bbc
to
3c97088
Compare
3c97088
to
a74fd69
Compare
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.
seems a better fix than what I had suggested I was going to do.
thank you! 鉂わ笍
(assuming you've tested this of course) sorry I'm unable to atm.
I tested this on a config which has the following: services.navidrome = {
enable = true;
openFirewall = true;
settings = {
Address = "0.0.0.0";
MusicFolder = "/run/navidrome/media/music";
};
}; and verified that the default port is in fact preserved, while the user-set settings are merged in:
|
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.
This breaks the nixos test, because settings is now required but unset by default. |
This will not affect the options, but allows to not configure settings at all, if you don't need to. Unbreaks the NixOS test, which relies on this behavior. Fixes: NixOS#312757
This will not affect the options, but allows to not configure settings at all, if you don't need to. Unbreaks the NixOS test, which relies on this behavior. Fixes: NixOS#312757
Description of changes
Fixes #312749. This fixes merging user-set values with default values.
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 馃憤 reaction to pull requests you find important.