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
shairport-sync: 3.3.9 -> 4.1.1 #203869
shairport-sync: 3.3.9 -> 4.1.1 #203869
Conversation
0494a80
to
eb7363d
Compare
all fixed @SuperSandro2000 |
shairport-sync 4.1.1 is out -- I was just about to make a PR myself until I saw this.
Is the latest sha if you want to update |
ebba8dd
to
5882a15
Compare
5882a15
to
813469b
Compare
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-ready-for-review/3032/1733 |
813469b
to
9722f82
Compare
If this could be merged that would be great! I'm looking to try the new pipewire backend. |
If you mention it on the discourse for already reviewed might help it get merged. Been through a few reviews already so should be good ot go (also you can apply the patch yourself if you want). @hexagonal-sun |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
Description of changes
Update shairport-sync from 3.9 to 4.1.1. Additionally it is now formatted and enables configuration using function arguments. Differences from the previous version are enabling jack and pipewire backends by default. Airplay 2 is disabled by default to not break the nixos module as running it is much more complicated (requires nqptp). Added myself as a maintainer.
Closes #198387
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)nixos/doc/manual/md-to-db.sh
to update generated release notesBuilt each
enableXXX
option successfully and ran the--help
command. Ran nixpkgs default build successfully with the pipewire backend. I have not tested the existing nixos module.Pinging @LnL7