Package: fdpowermon
Version: 1.5
Severity: normal

When my battery is full, fdpowermon still shows "Charging, 98%".
That's because the regexp that extracts the data from the acpi -b
output doesn't match the full status: "Battery 0: Full, 100%".
(This is probably also why removing the battery still shows the old
info, instead of reflecting the current state... see #676213.
The regexp doesn't match, so it just loops without doing anything.)

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. Fall back to acpi -b
if the system has an unexpected /sys/class/power_supply/ layout.

The following patch fixes it for me.

thanks,
Paul

--- /usr/bin/fdpowermon 2012-01-11 01:02:58.000000000 +0100
+++ /tmp/fdpowermon     2013-10-15 18:48:07.701382173 +0200
@@ -25,7 +25,7 @@
    open my $acpi, "acpi -b |";
    while ($acpi_output = <$acpi>) {
        chomp $acpi_output;
-               if ($acpi_output =~ /^Battery (\d): 
((Dis)?[Cc]harging|Unknown), ((\d)+)%(, ([\d:]*))?/) {
+               if ($acpi_output =~ /^Battery (\d): 
((Dis)?[Cc]harging|Unknown|Full), ((\d)+)%(, ([\d:]*))?/) {
            $bat = $1;
            $state = $2;
            $level = $4;


-- System Information:
Debian Release: jessie/sid
  APT prefers unstable
  APT policy: (500, 'unstable'), (500, 'stable'), (500, 'oldstable')
Architecture: amd64 (x86_64)

Kernel: Linux 3.9.5-vs2.3.6.5.wurtel-ws (SMP w/8 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash


-- 
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