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
zigbee2mqtt: 1.14.1 -> 1.14.2 #94583
Conversation
Thanks, rebased. |
The tests seem to have been skipped
Anyone has a clue why? |
|
Spot on. |
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 update script is a nice improvement 👍
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.
Looks good!
Thanks everyone! I ordered my CC2652RB USB-Stick just today, so I can't wait to set this up! |
@@ -1,9 +1,9 @@ | |||
{ pkgs, stdenv, system, dataDir ? "/opt/zigbee2mqtt/data" }: | |||
{ pkgs, stdenv, system, dataDir ? "/opt/zigbee2mqtt/data", nixosTests }: |
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.
@sweber83 Any particular reason we save state to /opt/zigbee2mqtt
and not /var/lib/zigbee2mqtt
?
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 default dataDir of the package conforms with the documentation of zigbee2mqtt, for non-NixOS nix users.
The NixOS module has its own default, which is /var/lib/zigbee2mqtt
.
We could change the package to also default to /var/lib/zigbee2mqtt
, but I think it makes sense to use the upstream default for non-NixOS distros.
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.
Thanks for the explanation.
Motivation for this change
1.14.2 adds support for https://www.tindie.com/products/slaesh/cc2652-zigbee-coordinator-or-openthread-router/.
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)