Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 398823 - app-admin/conky-1.8.1 - Add support to read POWER_SUPPLY_POWER_NOW
Summary: app-admin/conky-1.8.1 - Add support to read POWER_SUPPLY_POWER_NOW
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal enhancement (vote)
Assignee: Daniel Pielmeier
URL: http://git.omp.am/?p=conky.git;a=comm...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-01-13 23:35 UTC by nvinson234
Modified: 2012-01-16 20:06 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Patch to read POWER_SUPPLY_POWER_NOW (conky-1.8.1-battery_time.patch,655 bytes, patch)
2012-01-13 23:36 UTC, nvinson234
Details | Diff
Updated ebuild (conky-1.8.1-r6.ebuild,4.07 KB, text/plain)
2012-01-13 23:42 UTC, nvinson234
Details

Note You need to log in before you can comment on or make changes to this bug.
Description nvinson234 2012-01-13 23:35:16 UTC
A commit to conky's source was made so that POWER_SUPPLY_POWER_NOW would be read from /sys/class/power_supply/<battery name>/uevent in the event POWER_SUPPLY_CURRENT_NOW was not available.

According to http://git.omp.am/?p=conky.git;a=commit;h=c5d065d1199a0ef3b0d7f72368fac47d4579b901, POWER_SUPPLY_CURRENT_NOW was renamed to POWER_SUPPLY_POWER_NOW.  However, that commit was made just over a year ago and was not included in the conky-1.8.1 release.

Thus, the request is to apply this fix to conky-1.8.1, so that conky works correctly with modern kernels.

Reproducible: Always

Actual Results:  
conky reports battery time remaining is unknown if POWER_SUPPLY_CURRENT_NOW is missing from /sys/class/power_supply/BAT0/uevent.  According to other sources POWER_SUPPLY_CURRENT_NOW was renamed to POWER_SUPPLY_POWER_NOW.

Expected Results:  
conky report the remaining time to [dis]charge a battery.
Comment 1 nvinson234 2012-01-13 23:36:27 UTC
Created attachment 298897 [details, diff]
Patch to read POWER_SUPPLY_POWER_NOW
Comment 2 nvinson234 2012-01-13 23:42:07 UTC
Created attachment 298901 [details]
Updated ebuild
Comment 3 Daniel Pielmeier gentoo-dev 2012-01-16 20:05:59 UTC
Brenden any chance to get 1.8.2 out as stuff like this piles up here.
Comment 4 Daniel Pielmeier gentoo-dev 2012-01-16 20:06:12 UTC
+*conky-1.8.1-r6 (16 Jan 2012)
+
+  16 Jan 2012; Daniel Pielmeier <billie@gentoo.org> -conky-1.8.1-r5.ebuild,
+  +conky-1.8.1-r6.ebuild, +files/conky-1.8.1-battery-time.patch:
+  Fix bug #398823. Thanks to nvinson for the report and patch.