| 14 Jun 2021 |
dotlambda | I'm not too invested but if you make a PR upstream, feel free to add test_form_invalid_auth and test_form_cannot_connect from wallbox's tests. | 22:43:33 |
@hexa:lossy.network | https://github.com/NixOS/nixpkgs/pull/126895 | 23:27:30 |
| 15 Jun 2021 |
Zhaofeng Li | Actually, someone should DM balloob/home_assistant to let them know about the HN thread. I think it's only fair and they don't seem to be aware. | 01:11:30 |
@hexa:lossy.network | I would be surprised if they hadn't discussed that internally yet | 01:13:06 |
@hexa:lossy.network | pretty sure that is what turned the reaction around | 01:13:47 |
@hexa:lossy.network | https://nixos.wiki/wiki/index.php?diff=5894&oldid=5892 | 02:11:59 |
@hexa:lossy.network | first draft of a limited support passage | 02:12:18 |
Mic92 (Old) | I won't do anything right now in this matter over the couple next week. I have other priorities right now. | 07:05:09 |
@hexa:lossy.network | https://nixos.wiki/wiki/Home_Assistant#Running_a_recent_version_using_an_overlay | 13:07:54 |
@hexa:lossy.network | untested, but should work | 13:08:13 |
@hexa:lossy.network | Outdated dependencies
┏━━━━━━━━━━━━━━━┳━━━━━━━━━━━━┳━━━━━━━━┓
┃ Package ┃ Current ┃ Wanted ┃
┡━━━━━━━━━━━━━━━╇━━━━━━━━━━━━╇━━━━━━━━┩
│ numpy │ 1.20.2 │ 1.20.3 │
│ pydroid-ipcam │ 2021-06-01 │ 0.8 │
│ pyotgw │ 2021-03-25 │ 1.1b1 │
│ sqlalchemy │ 1.3.23 │ 1.4.13 │
└───────────────┴────────────┴────────┘
| 13:22:37 |
@hexa:lossy.network | I tried to expose the python package set post-overrides in passthru | 13:22:57 |
@hexa:lossy.network | and then use that as the package set checked in parse-requirements.py | 13:23:09 |
@hexa:lossy.network | diff --git a/pkgs/servers/home-assistant/default.nix b/pkgs/servers/home-assistant/default.nix
index c2eae375658..805306a2d9c 100644
--- a/pkgs/servers/home-assistant/default.nix
+++ b/pkgs/servers/home-assistant/default.nix
@@ -799,6 +799,7 @@ in with py.pkgs; buildPythonApplication rec {
tests = {
inherit (nixosTests) home-assistant;
};
+ pythonPackages = py;
};
meta = with lib; {
diff --git a/pkgs/servers/home-assistant/parse-requirements.py b/pkgs/servers/home-assistant/parse-requirements.py
index 27de7a60f37..1eeb0f1356c 100755
--- a/pkgs/servers/home-assistant/parse-requirements.py
+++ b/pkgs/servers/home-assistant/parse-requirements.py
@@ -32,7 +32,7 @@ from rich.console import Console
from rich.table import Table
COMPONENT_PREFIX = "homeassistant.components"
-PKG_SET = "python3Packages"
+PKG_SET = "home-assistant.pythonPackages"
# If some requirements are matched by multiple Python packages,
# the following can be used to choose one of them
| 13:24:55 |
@hexa:lossy.network | looks to be working, but parse-requirements doesn't yet find all packages again | 13:25:11 |
@hexa:lossy.network | * diff --git a/pkgs/servers/home-assistant/default.nix b/pkgs/servers/home-assistant/default.nix
index c2eae375658..805306a2d9c 100644
--- a/pkgs/servers/home-assistant/default.nix
+++ b/pkgs/servers/home-assistant/default.nix
@@ -799,6 +799,7 @@ in with py.pkgs; buildPythonApplication rec {
tests = {
inherit (nixosTests) home-assistant;
};
+ pythonPackages = py.pkgs;
};
meta = with lib; {
diff --git a/pkgs/servers/home-assistant/parse-requirements.py b/pkgs/servers/home-assistant/parse-requirements.py
index 27de7a60f37..1eeb0f1356c 100755
--- a/pkgs/servers/home-assistant/parse-requirements.py
+++ b/pkgs/servers/home-assistant/parse-requirements.py
@@ -32,7 +32,7 @@ from rich.console import Console
from rich.table import Table
COMPONENT_PREFIX = "homeassistant.components"
-PKG_SET = "python3Packages"
+PKG_SET = "home-assistant.pythonPackages"
# If some requirements are matched by multiple Python packages,
# the following can be used to choose one of them
| 13:26:28 |
@hexa:lossy.network | nvm, works | 13:26:45 |
@hexa:lossy.network | https://github.com/home-assistant/core/releases/tag/2021.6.5 | 19:18:20 |
@hexa:lossy.network | new dependency in a minor release 😒 | 19:18:37 |
@hexa:lossy.network | https://github.com/NixOS/nixpkgs/pull/126984 | 19:36:36 |
| @hexa:lossy.network invited balsoft. | 20:11:20 |
| balsoft joined the room. | 20:11:37 |
balsoft | First of all, thank you all for packaging home assistant, it's awesome to blink my lights from NixOS :) | 20:25:57 |
balsoft | But, as you might have guessed, I came with some feedback re: the drama that happened. I think it would be better for the image of the Nix community if instead of immediately merging stuff without reaching some consensus or at least understanding and thus producing extra drama, we would just wait a couple of days and let things cool down. Obviously not stating this as fact, just my opinion you're free to disregard. | 20:28:29 |
balsoft | Although, to be fair, I applaud the way you handled the issue otherwise, calm, polite and constructive. | 20:29:16 |
balsoft | * Although, to be fair, I applaud the way you handled the issue otherwise: calm, polite and constructive. | 20:29:23 |
@hexa:lossy.network | Yes, we all learned a great deal on how to handle such an oddball in the future! | 21:03:41 |
| Linux Hackerman joined the room. | 23:42:43 |
| 16 Jun 2021 |
| CRTified joined the room. | 08:35:35 |
CRTified | Wow, I've somehow missed the whole debacle around that ambee package. Combined with HA moving away from YAML more and more, I'm really starting to hope that there's a sensible alternative available 🤔 I mean, most components are probably easy to move to "just MQTT", but the number of available integrations is hard to ignore. | 08:47:02 |