-
-
Notifications
You must be signed in to change notification settings - Fork 641
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
chore: enable full ENV support and remove --sqlite option #6286
Conversation
Co-authored-by: Stephan Pauxberger <stephan@blackbuild.com>
Ich glaube die Änderung funktioniert nicht, zumindest hier bei mir:
Exec mag dann nicht:
möglicher Workaround (ungetestet mangels Zeit heute)
|
Da müsst ihr mir helfen, ich bin da leider kein Experte und mir fehlt das Linux zum testen. Gerne PR mit Bugfix. |
Update: Lt. Proc ist die Variable dann auch im Environment des Prozesses. Update 2 Update 3
Diff: index 8f537ec0..a346c7be 100755
--- a/docker/bin/entrypoint.sh
+++ b/docker/bin/entrypoint.sh
@@ -16,7 +16,7 @@ if [ -f ${HASSIO_OPTIONSFILE} ]; then
else
if [ "${SQLITE_FILE}" ]; then
echo "starting evcc: 'EVCC_DATABASE_DSN=${SQLITE_FILE} evcc --config ${CONFIG}'"
- exec EVCC_DATABASE_DSN="${SQLITE_FILE}" evcc --config "${CONFIG}"
+ exec env EVCC_DATABASE_DSN="${SQLITE_FILE}" evcc --config "${CONFIG}"
else
echo "starting evcc: 'evcc --config ${CONFIG}'"
exec evcc --config "${CONFIG}" Läuft dann damit: Using config file: /config/evcc.yaml |
Pull-Request: Schau mal bitte, ob ich das richtig gemacht habe - ist der erste pull request |
/cc @pauxus @goebelmeier funktioniert das mit dem ENV im Servicefile?