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
alsa-lib: 1.1.9 -> 1.2.2 and new alsa conf packages #83155
Conversation
This should probably target the staging branch. |
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.
Tested on my System, works as expected.
runHook postInstall | ||
''; | ||
|
||
meta = with stdenv.lib; { |
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.
Also no maintainer.
runHook postInstall | ||
''; | ||
|
||
meta = with stdenv.lib; { |
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 is no maintainer
here, maybe add yourself?
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.
I was not sure if I should fill myself. But I will give it a try.
Motivation for this change
alsa-lib is behind upstream and other alsa tools in nixpkgs. This PR adds to new packages
alsa-ucm-conf
andalsa-topology-conf
to reflect upstream's split to multiple repositories. Their content is linked into alsa-lib$out/share/alsa
directory, where alsa searches them.It looks like new alsa-lib is needed by new sof drivers to fully support sound in newer intel notebooks like mine (tested with custom nixos build).
https://bugzilla.redhat.com/show_bug.cgi?id=1772498
https://bugzilla.kernel.org/show_bug.cgi?id=201251
I removed some obsolete seds chaning
u_int
touint
. Upstream has fixed this some time ago. And the last remaining is probaly not needed anymore too.Replaces #74102
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)