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
mkchromecast: init at 5872a246 #59280
mkchromecast: init at 5872a246 #59280
Conversation
4549dc3
to
ffa9317
Compare
pulseaudio vorbis-tools sox lame flac faac ffmpeg | ||
]; | ||
|
||
doCheck = false; |
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.
Why are the tests disabled? Include a comment in the expression explaining why the tests are disabled.
@@ -0,0 +1,27 @@ | |||
{ lib, fetchFromGitHub, buildPythonApplication, pulseaudio |
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.
use python3Packages
as argument instead
, vorbis-tools, sox, lame, flac, faac, ffmpeg }: | ||
buildPythonApplication rec { | ||
pname = "mkchromecast"; | ||
version = "5872a246f0610b74fc2b197eb02dc91b96fb68cc"; |
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.
Should be something human readable, like "unstable-2011-12-31" (change the date to the commit date of course)
Sorry for the duplicate. This was apparently already submitted and the PR is in a further stage than this one: #57033 |
Motivation for this change
This adds the mkchromecast package. i have tested this using:
And it would show my screen on my TV after a few moments.
The upgrade of pychromecast was not needed strictly speaking, but it seemed appropriate to enable all functionality of mkchromecast.
There wasn't a recent release, but that's something that hopefully will come (muammar/mkchromecast#244).
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)