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
ympd service: Allow webPort to be int #59362
Conversation
@@ -15,7 +15,7 @@ in { | |||
enable = mkEnableOption "ympd, the MPD Web GUI"; | |||
|
|||
webPort = mkOption { | |||
type = types.string; | |||
type = types.either types.string types.int; |
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.
There's types.port
even
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.
uuuh, nice!
serviceConfig.ExecStart = let | ||
port = builtins.toString cfg.webPort; | ||
in | ||
"${pkgs.ympd}/bin/ympd --host ${cfg.mpd.host} --port ${toString cfg.mpd.port} --webport ${port} --user nobody"; |
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.
And here you can just inline --webport ${toString port}
I'm ready to squash! |
@@ -15,7 +15,7 @@ in { | |||
enable = mkEnableOption "ympd, the MPD Web GUI"; | |||
|
|||
webPort = mkOption { | |||
type = types.string; | |||
type = types.either types.string types.port; # string for backwards compat |
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.
Oh actually one more thing, while you're changing this part of the code, you can also switch to types.str
, as types.string
is deprecated.
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.
馃憤
Feel free to squash |
Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
9dad17f
to
31884f7
Compare
Motivation for this change
I am lazy and don't want to type
"
. 馃槃I don't see a reason why this should not be allowed to be int.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)I did not yet test-build my system with this change.