Long story short, I'm running byobu in a Debian chroot on an Android system and noticed that the battery status notification doesn't work with the battery information this Android kernel spits out. In particular, /sys/class/power_supply/battery/uevent has fewer variables and POWER_SUPPLY_STATUS sometimes contains a space ("not charging") so sourcing breaks. For the record, this is on an HTC Wildfire with CyanogenMod 7.2 (Android 2.3.7, Linux 2.6.35-14).
So I fixed it, see attached patch (against trunk). Notable changes include:
* "uevent" is now read more smartly so that spaces don't break everything
* I've preferred POWER_SUPPLY_CAPACITY get the current charge in percent since both Android and an ACPI x86 notebook I've checked had that
* strings from "uevent" are now lowercased to accommodate for differences in that regard
* I added a configuration variable (BATTERY_SHOW_WHEN_FULL) to display the battery indicator even when the battery is full since I found the original behaviour slightly surprising (the default is still as before though)
I admit that I didn't test the patch on other devices, but it should support a superset of devices and implementations.
Hi Felix,
Thanks for the patch!
I did some basic testing here, and it seems that something in your code has changed the logic, and regressed the battery status support on at least my Thinkpad.
I'll take a closer look and see if I can see what exactly has failed.