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
Fix home-assistant netatmo support #60947
Conversation
(FYI: @dotlambda since you're hass maintainer) |
@GrahamcOfBorg build python2.pkgs.pyatmo python3.pkgs.pyatmo |
Please provide an appropriate |
Upstream has no tests for this package. |
In that case, please set |
Looking at long term maintenance, wouldn't it be better to keep the default checkPhase? If we do and upstream adds tests, it will either transparently work or require us to take further action at that time (disable or fix). Whereas if we doCheck = false it seems like there's potential for it slipping through a version bump unnoticed. I also don't see C++ autotools packages disabling the default checkPhase when "make check" does nothing, which is akin to this situation. </nitpicking> (but really I'm just trying to understand better if there's an established convention, and why) |
I've never really understood the convention, but it's what FRidh wants. |
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.
Please re-run parse-requirements.py
.
@dotlambda rebased onto 0.92 changes. GitHub never sent me an email notification about you changing the base branch of the PR so I was just waiting for 0.92 to get into master... (And GitHub sucks as usual and decided to put everyone as reviewer. Sorry for the spam.) |
@GrahamcOfBorg build python2.pkgs.pyatmo python3.pkgs.pyatmo |
Motivation for this change
The netatmo components for home-assistant are broken in current nixpkgs due to a missing dependency. That dependency (pyatmo) happens to not be packaged yet, so I'm adding it to nixpkgs as part of this PR.
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)