Skip to content
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

nixos/minidlna: add options for user and group #373604

Closed
wants to merge 1 commit into from

Conversation

earldouglas
Copy link
Member

Description of changes

This adds configuration options to specify the user/group to use for
running the MiniDLNA daemon.

The default minidlna user/group will not be created when a different
user/group is specified.

Things done
  • Built on platform(s)

    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested on NixOS (x86_64-linux) via nixos-rebuild -I path/to/nixpkgs

    • Using the default minidlna user/group
    • Using a custom user/group

This adds configuration options to specify the user/group to use for
running the MiniDLNA daemon.

The default `minidlna` user/group will not be created when a different
user/group is specified.

Co-authored-by: Arne Keller <[email protected]>
@github-actions github-actions bot added 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 8.has: module (update) This PR changes an existing module in `nixos/` labels Jan 14, 2025
@ofborg ofborg bot added the 2.status: merge conflict This PR has merge conflicts with the target branch label Jan 14, 2025
@earldouglas earldouglas deleted the minidlna-user-group branch January 14, 2025 01:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.status: merge conflict This PR has merge conflicts with the target branch 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 8.has: module (update) This PR changes an existing module in `nixos/`
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant