retitle 726525 should parse /sys directly, rather than calling acpi
severity 726525 wishlist
thanks

Op 16-10-13 14:40, Paul Slootman schreef:
> Package: fdpowermon
> Version: 1.5
> Severity: normal
> 
> When my battery is full, fdpowermon still shows "Charging, 98%".

This was fixed in fdpowermon 1.6 (which did not make stable only because
it missed the freeze due to my procrastination)

[...]
> Perhaps it would also be nicer to directly access the
> /sys/class/power_supply/* stuff?  That would prevent spawning
> a couple of processes every 3 seconds.

Yeah, I should probably do that. Polling really isn't a good use of
resources, even if we do it only once every three seconds.

The reason I didn't was lazyness, but that's really not a good enough
excuse. I'll look into this when next I have some time. For now,
retitling and changing severity so this part of the bug stands out ;-)

> Fall back to acpi -b
> if the system has an unexpected /sys/class/power_supply/ layout.

Since acpi itself does little more than reading files in that directory,
I doubt I'll have more luck in that case (unless there's a bug in my
code, of course, which should then be fixed). Since adding fallback code
would add complexity to fdpowermon _and_ result in code paths which
would see very little testing, I'll just drop that part instead.

-- 
This end should point toward the ground if you want to go to space.

If it starts pointing toward space you are having a bad problem and you
will not go to space today.

  -- http://xkcd.com/1133/


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to