isDipesh / gnome-shell-extension-sensors

Gnome shell extension: Shows CPU temperature, HDD temperature, voltage and fan RPM
https://motorscript.com/gnome-shell-extension-sensors/
161 stars 153 forks source link

Latest version on extensions.gnome.org crashes gnome #79

Closed beatgammit closed 11 years ago

beatgammit commented 11 years ago

gnome-shell: 3.8.2 OS: Arch Linux

The comments on the home page seem like the problem I'm seeing, but I haven't looked at the logs.

Let me know what other debug info I can give.

adrianbroher commented 11 years ago

Hello @beatgammit,

Seems like hendersj is on the right track there. To verify this could you please answer the following questions?

You said you use Arch, am I right about the fact that you use systemd as init system?

If that is the case, what is the output of

systemctl show hddtemp.service -p ActiveState

and

systemctl show hddtemp.service -p MainPID

beatgammit commented 11 years ago

Also:

systemctl status hddtemp.service

Loaded: error (Reason: No such file or directory)

I don't have hddtemp installed. I'll install and get back to you.

adrianbroher commented 11 years ago

I don't have hddtemp installed. I'll install and get back to you.

That's okay, exactly this case should be handled properly.

Could you please test the latest version, following the instructions for manual installation?

beatgammit commented 11 years ago

I installed hddtemp, started the server and still got the crash. I'll uninstall it and test the latest version.

beatgammit commented 11 years ago

That works for me! Temperatures look consistent with acpi -t, but I get two temp1 entries in the drop-down. I'm not sure what this is, but acpi -t only gives two:

$ acpi -t
Thermal 0: ok, 26.3 degrees C
Thermal 1: ok, 62.5 degrees C

Screenshot

adrianbroher commented 11 years ago

This extension doesn't use the interface acpi -t uses, it relies only on the sensors (like in the application 'sensors') output. So if there are two "temp1" entries in the sensors output it works as expected. If that isn't the case please open a new issue for that.