home-assistant-ecosystem / home-assistant-cli

:computer: Command-line tool for Home Assistant
Other
446 stars 69 forks source link

jsonpath-rw dependency conflicts with hass dependencies #322

Closed jjlawren closed 4 years ago

jjlawren commented 4 years ago

I'm not positive this is related to the version of Home Assistant, but I've found the jsonpath-rw dependency pulled in by CLI causes runtime conflicts with the HA rest component which depends on jsonpath: https://github.com/home-assistant/home-assistant/issues/32284.

stale[bot] commented 4 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

jjlawren commented 4 years ago

This appears to remain a valid issue.

fabaff commented 4 years ago

Yes, it's a valid issue. Was also, present for the Fedora package (but more that there jsonpath-rw was not up-to-date and released over 5 years ago for the last time).

Home Assistant is strictly pinning the release of the dependency. For hass-cli I soften them to allow the installation in the same venv. The pinning causes issues if a dependency is also pinning the requirements. This leads to a race condition during the initialization process.

I will look into it.

onkelbeh commented 4 years ago

Is something going on here? Sometimes I run into conflicts between jsonpath and jsonpath-rw HA's Androidtv component has issues if jsonpath-rw is installed.