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
python3Packages.zha-quirks: 0.0.45 -> 0.0.46 #104227
Conversation
, pytest }: | ||
|
||
buildPythonPackage rec { | ||
pname = "zha-quirks"; | ||
version = "0.0.45"; | ||
|
||
nativeBuildInputs = [ pytest ]; | ||
#nativeBuildInputs = [ pytest ]; |
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.
this shouldn't be needed, I would just remove it.
#nativeBuildInputs = [ pytest ]; |
, pytest }: | ||
|
||
buildPythonPackage rec { | ||
pname = "zha-quirks"; | ||
version = "0.0.45"; | ||
|
||
nativeBuildInputs = [ pytest ]; | ||
#nativeBuildInputs = [ pytest ]; | ||
buildInputs = [ aiohttp zigpy ]; |
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.
these are python packages, should be propagatedBuildInputs
buildInputs = [ aiohttp zigpy ]; | |
propagatedBuildInputs = [ aiohttp zigpy ]; |
If #104202 gets merged first, this could also be updated to 0.0.46 |
Can you please remove the cherry-pick reference? It does not seem like it is part of nixpkgs. |
65c8e99
to
6676c02
Compare
@mweinelt You are right, I put it in a wrong branch initially I updated the version too |
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.
tests aren't being ran
nativeBuildInputs = [ pytest ]; | ||
buildInputs = [ aiohttp zigpy ]; | ||
propagatedBuildInputs = [ aiohttp zigpy ]; | ||
checkInputs = [ pytest conftest asynctest ]; |
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.
checkInputs = [ pytest conftest asynctest ]; | |
checkInputs = [ pytestCheckHook conftest asynctest ]; |
{ stdenv, buildPythonPackage, fetchPypi | ||
, aiohttp, zigpy | ||
{ lib, buildPythonPackage, fetchPypi | ||
, aiohttp, zigpy, conftest, asynctest | ||
, pytest }: |
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.
, pytest }: | |
, pytestCheckHook }: |
Also unbreak the tests
6676c02
to
f4347d6
Compare
Looks like it also needs |
https://github.com/zigpy/zha-device-handlers/blob/dev/requirements_test_all.txt |
Result of 2 packages built:
|
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.
LGTM
https://github.com/NixOS/nixpkgs/pull/104227
2 packages built:
python37Packages.zha-quirks python38Packages.zha-quirks
(cherry picked from commit f2e91df5a0b2c187f3b66579098d476f0b48089a)
Motivation for this change
Some dependencies were missing for the tests: conftest and asynctest
Things 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)