-
-
Notifications
You must be signed in to change notification settings - Fork 15.1k
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
Add "Lightweight Music Server" application + service #147206
Conversation
Note: "Checking EditorConfig / tests" fails with "Wrong indent style found (tabs instead of spaces)" because tabs are in the patch included. |
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.
Thanks for your contribution. I have left a few suggestions for the NixOS module that I hope you find helpful. Please don't hesitate to reach out for any reason.
hi @aanderse, thanks for the reviewing, all very good suggestions. Where can I read up about library functions such as |
Note: there is one strange issue. The service configuration throws a strange error if I use the INI format instead of toml here (i.e. (Please ignore the line numbers, as they might have changed due to whitespace added.)
|
I can't figure out why... but toml works, so the PR is fine as is. |
5135605
to
e3aee13
Compare
e3aee13
to
b8b238c
Compare
It might be worth closing this PR and creating a new one so we don't spam all the people who accidentally were subscribed to this. |
Motivation for this change
It's a nice efficient music streaming server, especially suitable for resource constrained devices / servers. See https://github.com/epoupon/lms.
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)nixos/doc/manual/md-to-db.sh
to update generated release notes