search

Alexandre-io / homeassistant-vcontrol

Vcontrold Home Assistant add-on repository
Apache License 2.0
41 stars 25 forks source link

Fix error code check #17

Closed ppuetsch closed 1 year ago

ppuetsch commented 1 year ago

Hi @Alexandre-io ,

another bug: On my setup, vcontrold sets $E1... to "OK" if the command is succesful. So the check [ "x$E'$i'" = x ] will be substituted to [ "xOK" = x ] which is of course never true.

With the below fix, I do have a running setup again.

I am not sure whether my vcontrold (remote one) is in some sense special here - so if the below bugfix is actually only a fix for me - or whether its a general bugfix.

Alexandre-io commented 1 year ago

Hi @ppuetsch ,

You are right! I used the vcontrold example but there is glitch. (https://github.com/openv/vcontrold/blob/master/doc/examples/rrdb/rrdb-update.tmpl#L1)

Thanks!