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

alsa-lib: 1.1.9 -> 1.2.2 and new alsa conf packages #83155

Merged
merged 3 commits into from Apr 5, 2020

Conversation

roastiek
Copy link
Contributor

Motivation for this change

alsa-lib is behind upstream and other alsa tools in nixpkgs. This PR adds to new packages alsa-ucm-conf and alsa-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 to uint. Upstream has fixed this some time ago. And the last remaining is probaly not needed anymore too.

Replaces #74102

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.

@Emantor
Copy link
Member

Emantor commented Mar 28, 2020

This should probably target the staging branch.

Copy link
Member

@Emantor Emantor left a 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; {
Copy link
Member

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; {
Copy link
Member

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?

Copy link
Contributor Author

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.

@roastiek roastiek changed the base branch from master to staging March 29, 2020 18:32
@FRidh FRidh mentioned this pull request Apr 3, 2020
@FRidh FRidh added this to WIP in Staging via automation Apr 3, 2020
@FRidh FRidh moved this from WIP to Needs review in Staging Apr 3, 2020
@FRidh FRidh merged commit 98cefdd into NixOS:staging Apr 5, 2020
Staging automation moved this from Needs review to Done Apr 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Staging
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants