BrainicHQ / tado-assistant

Enhance your Tado experience with this free, user-friendly auto-assist tool, now featuring multi-account support! Ideal for both tech-savvy and non-technical users, it offers advanced home state monitoring, smart adjustments, and open window detection. Independently developed and not affiliated with Tado GmbH.
GNU General Public License v3.0
21 stars 3 forks source link

Multi Arch Support #2

Closed saschabrockel closed 9 months ago

saschabrockel commented 9 months ago

I wanted to try your work but got the following error:

WARNING: The requested image's platform (linux/arm64/v8) does not match the detected host platform (linux/amd64) and no specific platform was requested
32fb0b3145eabfc1fff1890a77f7abbc3ef826a941cf2c5e2275c36b39fa2f9c

It's really important to support multiple platforms. And linux/amd64 is the most basic one. Do you know how to do it? I lost your mails...

s1lviu commented 9 months ago

Hi @saschabrockel,

Please check now. I had done the build on my M1 mac and that's why the problem occurred.

Thanks!

saschabrockel commented 9 months ago

Seems to work but I get /usr/local/bin/tado-assistant.sh: line 51: username_var: invalid indirect expansion as error in the logs of the container now. Does the username_var needs to be written in ' ' ?

s1lviu commented 9 months ago

No, it seems was a bash issue. Should be fixed now: https://github.com/BrainicHQ/tado-assistant/commit/d3ec26e61bc92c5ce03b619e10d111da953014c2

saschabrockel commented 9 months ago

Works now!

saschabrockel commented 9 months ago

Just one idea. It would be awesome if the timezone could be adjusted. I guess the default is UTC because it is 1 hour back for me. I tried the TZ env variable but it does not work.