-
-
Notifications
You must be signed in to change notification settings - Fork 501
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
DietPi-Software | Home Assistant: Further install enhancements and add workarounds for external bugs #3231
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
+ DietPi-Software | Home Assistant: Skip pyenv virtualenv. pyenv itself is installed locally for the "homeassistant" user already, hence another virtualenv is overkill. This simplifies install and pyenv activation and reduces disk usage overhead. + DietPi-Software | Home Assistant: Add workaround to install home-assistant-frontend prior to first HA start: home-assistant/core#28361 (comment) + DietPi-Software | Home Assistant: Install aiohttp_cors manually as well due to similar error faced with Python 3.8.0 outside of virtualenv
MichaIng
changed the title
v6.27
DietPi-Software | Home Assistant: Further install enhancements and add workarounds for external bugs
Nov 15, 2019
+ DietPi-Software | Home Assistant: Install workaround Python modules with specific required version as otherwise colliding dependency versions can be pulled which in cases break further modules install through the Home Assistant web UI. + DietPi-Software | Home Assistant: Massively reduce dependencies to a minimum of current requirements to succeed build and eliminate all warnings as well. Do not install SQLite, FFmpeg and MariaDB support by default, which is not used by our setup: SQLite binary is only required if one wants to access the default SQLite database directly. FFmpeg for AV integrations. MariaDB as SQLite replacement generally makes sense for larger setup, but must be setup by user manually currently anyway. We'll add some info to our online docs about how to add additional Python modules into the Home Assistant pyenv.
+ DietPi-Software | Home Assistant: Python v3.8.0 does work perfectly well and aiohttp_cors workaround is not required. The service only needs to be restarted one time, if web UI access fails or if it contains a notification about certain components were unable to load. For some reason it by times fails to install certain Python modules ondemand after being started the first time. We'll add an info about this to our online docs. + DietPi-Software | Home Assistant: Since some components and setups (e.g. MariaDB for database) require additional Python modules or even Python build dependencies, it makes sense to allow users maintaining a custom deps list via dietpi.txt. This allows easy reinstalls and server migrations without the need to recheck, run pip installs manually and in cases redo the long taking Python build if some developer library was missing for a certain HA component/integration. + DietPi-Software | Home Assistant: Move the pyenv activation into its own script, to be sourced from originating script/shell. Check for correct user navigate to its home, which assures correct Python version to be loaded automatically as well.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Status: Ready
Reference: #2374
Commit list/description: