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
pythonPackages.{pure-python-adb,adb-shell,androidtv}: init #98452
Conversation
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.
Off to a good start, but does not look complete yet.
8a93af6
to
9756320
Compare
We're still missing androidtv and pure-python-adb. The rest looks good. |
I was planning on opening separate PRs for those. Is that okay? |
Just include them here please, so we can have one pr including the component package regeneration. |
9756320
to
f2a15b2
Compare
I've added all the required packages. I'll make the changes to home-assistant in a followup PR. Home assistant needs a version bump anyway as |
62485a5
to
1b13a04
Compare
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.
The packages look pretty good, please disable adb-shell
on py27 and regenerate the component packages for home-assistant.
I've taken care of the home-assistant update in #99217. You can regenerate the component-packages now and I'll rebase on top of that, or we do it the other way around. I don't mind. |
I think I'll just wait until your PR goes in, and then I'll rebase on top of that. I'm in no rush to get this merged. |
Result of 6 packages built:
|
# Disable tests as they require docker, docker-compose and a dedicated | ||
# android emulator | ||
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.
Whoops, we're missing a pythonImportsCheck
here.
Last gotcha.
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.
Last nit.
Motivation for this change
Support
androidtv
in Home AssistantThings 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)