-
-
Notifications
You must be signed in to change notification settings - Fork 14k
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
Update pulseaudio module #270677
Update pulseaudio module #270677
Conversation
4ddac0e
to
4aa835b
Compare
662be63
to
5776d5d
Compare
I know, it sounds weird to give a home directory 755 permissions but this is actually intended by pulseaudio as you can see here: |
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.
Reword the commit message:
nixos/pulseaudio: set permission of pulse home directory
Closes #114399
5776d5d
to
5b2714a
Compare
5b2714a
to
d14a788
Compare
Rebased and merged. Thank you for your contribution! |
Description of changes
-> The start-script of the pulseaudio daemon changes the permission to 755, but this gets reset on rebuild (with changes not regarding pulseaudio, so the daemon is not restarted)
-> with the
homeMode
option, this gets set to 755 on rebuild-> Fixes Issue, described in #114399
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/
)