$acpiacadapter checks for /sys/class/power_supply/AC/uevent, which is not present and just puts "no ac_adapters?" in the UI, the correct path is /sys/class/power_supply/ADP1/uevent
Created attachment 174611 [details, diff] patch to fix the ac adapter path even though the comment in linux.c says it's AC in 2.6.25+, it's always been ADP1 on this system, currently running 2.6.28-rc7 - you might want to ask others first though to be sure
Comment on attachment 174611 [details, diff] patch to fix the ac adapter path fixed a typo in the patch's filename, this is with conky-1.6.2_pre btw
So has this path changed? Or was it always wrong? I don't want to break things for older kernels.
well, it's always been ADP1 on this lappy, i.e. 2.6.26-2.6.28
there's actually an even better way to fix this, acpiacadapter could have an option like the battery_* options, that one might even default to AC, but I can change it to ADP1 here
Any news on this. Still a problem?
Conky: can't open /sys/class/power_supply/AC/uevent: No such file or directory % ls /sys/class/power_supply ADP1@ BAT0@ [I] app-admin/conky (1.7.2-r2@11/04/2009): An advanced, highly configurable system monitor for X yes, still a problem
(In reply to comment #5) > there's actually an even better way to fix this, acpiacadapter could have an > option like the battery_* options, that one might even default to AC, but I can > change it to ADP1 here > Okay after some research it looks like this is not kernel dependent, but hardware dependent. So the name of the adapter may be AC but could also have other values for some hardware. @brenden is it possible to implement something like christoph suggests here. So the user can provide the name of the adapter and conky looks for /sys/class/power_supply/${var}
1.8.0 will have an aditional argument of $acpiacadapter, which can be used to specify the subfolder of /sys/class/power_supply/ containing the info. The patch is in <http://git.omp.am/?p=conky.git;a=commit;h=f5363b6539d7bd4928437823d3ebc81b7f7502d5> (although it probably doesn't apply on 1.7.2 cleanly) I would appreciate some feedback, as i don't have the hardware to test it on. thanks.
Created attachment 217322 [details] conky-9999.ebuild Christoph you can use the attached live ebuild so you don't have to mess with patches. :)
it actually does work, ty :>
(In reply to comment #11) > it actually does work, ty :> Good to hear :) @pavelo: As this is also fixed in the upcoming 1.8 series. Does it make sense to bump to 1.8.0_rc2? Or will there be a final version of 1.8.0 or 1.8.1 in the not to distant future.
(In reply to comment #11) > it actually does work, ty :> My pleasure ;) (In reply to comment #12) > @pavelo: As this is also fixed in the upcoming 1.8 series. Does it make sense > to bump to 1.8.0_rc2? Or will there be a final version of 1.8.0 or 1.8.1 in the > not to distant future. I couldn't say. I hope it's not too far down the road.
I have just committed conky-1.8.0_rc2 which should fix this issue.
