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.16.1 -> 1.16.2 #105973
zigbee2mqtt: 1.16.1 -> 1.16.2 #105973
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.
Good catch regarding the DATADIR. I have a question and a suggestion.
@GrahamcOfBorg test zigbee2mqtt |
pkgs/servers/zigbee2mqtt/default.nix
Outdated
mkdir $out/bin | ||
ln -s $out/lib/node_modules/zigbee2mqtt/index.js $out/bin/zigbee2mqtt | ||
ln -s $out/lib/node_modules/zigbee2mqtt/cli.js $out/bin/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.
any reason not to use upstream generated binary? that way the chmod, rm,mkdir and ln commands would be come obsolete.
or is upstream bin/zigbee2mqtt symlink not working afterwards?
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.
nit: s/binary/executable/
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.
Good point. I removed the postInstall
completely and the NixOS test is running just fine.
Btw: I actually tweaked the |
Yeah, that was the reason why I didn't update z2m to 1.16.2 already. Created an upstream issue over at svanderburg/node2nix#213. |
Result of 1 package 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.
Deployed this version of zigbee2mqtt on my server at home and the network got kept alive!
Motivation for this change
https://github.com/Koenkk/zigbee2mqtt/releases/tag/1.16.2
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)/cc @mweinelt @sweber83