Page MenuHomeFreeBSD

sysutils/apcupsd: Add patch to fix APC BE600M1 reporting
ClosedPublic

Authored by dereks_lifeofadishwasher.com on Aug 31 2020, 11:50 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Nov 25, 11:44 PM
Unknown Object (File)
Sat, Nov 23, 5:56 PM
Unknown Object (File)
Fri, Nov 22, 7:26 PM
Unknown Object (File)
Thu, Nov 21, 5:41 AM
Unknown Object (File)
Tue, Nov 19, 10:22 PM
Unknown Object (File)
Tue, Nov 19, 11:06 AM
Unknown Object (File)
Sun, Nov 10, 2:56 AM
Unknown Object (File)
Tue, Nov 5, 9:31 AM
Subscribers
None

Details

Summary

Reading here [1] and using the patch mentioned here [2] I was able to
get apcupsd running on 12.1 to correctly report data from a BE600M1.

Also, some functions in apctest now work as well.

[1] https://sourceforge.net/p/apcupsd/mailman/apcupsd-users/thread/CAMe0L0Vc9BzCYehm-bZUoPgHaL-43xsjetQ3hi7gD380b7NsqQ%40mail.gmail.com/#msg36287119
[2] http://apcupsd-ups-control-software.10985.n7.nabble.com/Potential-bug-with-USB-driver-and-HID-reports-td10001.html#a10043

Testing current apcupsd reporting via apcaccess had limited data. With the 2017 patches [1] this appears to have fixed this issue with USB I/O issues.
I don't know how this patch plays with other APC USB attached UPS. I can't fully vet the cpp from the patches but it at least fixes my issue with the BE600M1.

Test Plan

Build with patch, see apcaccess reports more data, continue to run apcupsd without errors.

Diff Detail

Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 33405
Build 30703: arc lint + arc unit

Event Timeline

Turn this into an optional patch with an addition to pkg-message

Thanks for the patch, very appreciated.

This revision is now accepted and ready to land.Sep 4 2020, 11:28 PM