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/pulseaudio: Add extraModules config option #48639
Conversation
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: pulseaudio-modules-bt Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: pulseaudio-modules-bt Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: pulseaudio-modules-bt Partial log (click to expand)
|
cb39d79
to
83f13fc
Compare
83f13fc
to
6f28c77
Compare
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: pulseaudio-modules-bt Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: pulseaudio-modules-bt Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: pulseaudio-modules-bt Partial log (click to expand)
|
I can't cherry-pick on nixos-unstable so I haven't tested it but I am all for it, the current mechanism to include the extra bluetooh module does not feel clean. |
This fixes a segmentation fault when loaded via pulseaudio's dl-search-path
6f28c77
to
78c0e1a
Compare
Success on x86_64-linux (full log) Attempted: pulseaudio-modules-bt Partial log (click to expand)
|
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: pulseaudio-modules-bt Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: pulseaudio-modules-bt Partial log (click to expand)
|
Motivation for this change
Previously the method to use
pulseaudio-modules-bt
was very hacky.With this PR the mechanism to use out-of-tree PA modules becomes:
The caveats are that any derivations in
extraModules
is expected to follow the same directory structure aspulseaudio
does internally, that is:"lib/pulse-${pulseaudio.version}/modules/module.so"
and the derivation is expected to take an argument
pulseaudio
that is the package specified inhardware.pulseaudio.package
.Any
extraModules
are built with the specified package as an override.Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)cc @teto @lovek323
edit: Additional documentation for any archeologists.