[Nut-upsdev] cyberpower driver

[Arjen de Korte]

> Arjen, I installed the latest version of the trunk, and I am having some
> trouble with the newest powerpanel driver:
>
> Network UPS Tools -  CyberPower text/binary protocol UPS driver 0.20.7 ()
> Warning: This is an experimental driver.
> Some features may not function correctly.
>
> debug level is '4'
> Trying binary protocol...
> send: (2 bytes) => 0d 0d
> read: timed out
> send: (2 bytes) => 46 0d
> read: timed out
> send: (2 bytes) => 46 0d
> read: timed out
> send: (2 bytes) => 46 0d
> read: timed out
> Trying text protocol...
> send: (2 bytes) => 0d 0d
> read: (2 bytes) => 23 32
> send: (3 bytes) => 50 34 0d
> read: (46 bytes) => 23 42 43 31 32 30 30 20 20 20 20 2c 31 2e 36 30 30 2c
> 30 30 30 30 30 30 30 30 30 30 30 30 2c 43 59 42 45 52 20 50 4f 57 45 52 20
> 20 20 20

Oops! Please change line 733 from

                if (ret < 47) {
to
                if (ret < 46) {

Apparently I didn't count the number of bytes we expect correctly.
Previously, we only checked if we received at least one by the way, so
this was never an issue before.

> I not sure exactly what you've changed, but I am willing to start
> testing...

No need for that, the debug output is sufficient. If you want to, feel
free to submit this change (I don't have SVN access from where I'm now).

Best regards, Arjen