pythonPackages.haas-nabucasa: init at 0.29#75876
Conversation
a05d36f to
47a7785
Compare
jonringer
left a comment
There was a problem hiding this comment.
diff LGTM
commits LGTM
has tests
[4 built, 2 copied (0.0 MiB), 0.0 MiB DL]
https://github.com/NixOS/nixpkgs/pull/75876
2 package were built:
python37Packages.snitun python38Packages.snitun
just some small suggestions
|
@GrahamcOfBorg build python37Packages.snitun python38Packages.snitun |
|
@jonringer I added another package to this pr since it depends on the original one, or should I open a new pull request for the next one? |
jonringer
left a comment
There was a problem hiding this comment.
your commit history should be:
pythonPackages.snitun: init at 0.20
pythonPackages.haas-nabucasa: init at 0.29
I would recommend using git rebase -i
|
it's fine, just need to update your PR to describe what you're doing |
8b56dbc to
9cceab6
Compare
|
you have snitun changes in the pythonPackages.haas-nabucasa commit do the following to fix: |
9cceab6 to
7b4d5a0
Compare
pythonPackages.haas-nabucasa: init at 0.29 (cherry picked from commit 45e69f1)
|
@Scriptkiddi can this be moved inside the As elaborated in NabuCasa/hass-nabucasa#119 (comment), upstream "doesn't care about other ways to run Home Assistant" (except docker and virtualenv install), so we should probably not have it in the curated python package set, but move it to https://github.com/NixOS/nixpkgs/blob/master/pkgs/servers/home-assistant/default.nix, (with some overrides, so this module doesn't block |
|
I will do that, I will also see if we can not remove such a module upstream from the hass cloud module so its not required, lets see how good my chances are seeing that those guys from nabucasa started hass .... |
Motivation for this change
Things done
nix-shell -p nix-review --run "nix-review wip"sandboxinnix.confon non-NixOS linux)./result/bin/)nix path-info -Sbefore and after)Notify maintainers
cc @