On a HP Compaq nc6120 laptop, the battery plugin does neither display the temperature nor the fan status. Attached patch should fix the problem. Reproducible: Always Steps to Reproduce: 1. Start battery panel plugin Actual Results: Temperature and fan status are not shown. Expected Results: Temperature and fan status being displayed. Note: libacpi.c contains lines like the following: proc_fan_status="/proc/acpi/fan/*/state"; if ( (fp=fopen(proc_fan_status, "r")) == NULL ) return 0; This cannot work since fopen does not expand wildcards.
Created attachment 986 Proposed bugfix (patch for panel-plugin/libacpi.c)
Why would a *battery* plugin display the fan or temperature status?
(In reply to comment #2) > Why would a *battery* plugin display the fan or temperature status? Of course, that is not mandatory. But if the application contains the feature, then it should work and not be broken on some hardware platforms.
> But if the application contains the feature, then it should work and not be > broken on some hardware platforms. In fact it is broken on any ACPI hardware. To quote from the corresponding Gentoo bug <http://bugs.gentoo.org/165989>: "Please note that this bug should be present on _any_ (ACPI) hardware: The plugin tries to fopen a file "/proc/acpi/fan/*/state" which cannot work, since fopen of course doesn't expand wildcards. This is a regression from the 0.3.0 version."
Patch applied. Thanks