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 150 forks source link

The Settings page does not work #100

Open svu opened 10 years ago

svu commented 10 years ago

In the GNOME Shell Extension Preferences, I see:

TypeError: malformed UTF-8 character sequence at offset 51

Stack trace: SensorsPrefsWidget<._getHddTempLabels@/usr/share/gnome-shell/extensions/temperature@xtranophilist/prefs.js:189 wrapper@/usr/share/gjs-1.0/lang.js:213 SensorsPrefsWidget<._init@/usr/share/gnome-shell/extensions/temperature@xtranophilist/prefs.js:117 wrapper@/usr/share/gjs-1.0/lang.js:213 buildPrefsWidget@/usr/share/gnome-shell/extensions/temperature@xtranophilist/prefs.js:242 Application<._selectExtension@/usr/share/gnome-shell/js/extensionPrefs/main.js:100 wrapper@/usr/share/gjs-1.0/lang.js:213 Application<._extensionSelected@/usr/share/gnome-shell/js/extensionPrefs/main.js:119 wrapper@/usr/share/gjs-1.0/lang.js:213 Application<._selectExtension@/usr/share/gnome-shell/js/extensionPrefs/main.js:110 wrapper@/usr/share/gjs-1.0/lang.js:213 Application<._extensionsLoaded@/usr/share/gnome-shell/js/extensionPrefs/main.js:219 wrapper@/usr/share/gjs-1.0/lang.js:213 _emit@/usr/share/gjs-1.0/signals.js:124 ExtensionFinder<._extensionsLoaded@/usr/share/gnome-shell/js/misc/extensionUtils.js:178 wrapper@/usr/share/gjs-1.0/lang.js:213 done@/usr/share/gnome-shell/js/misc/fileUtils.js:33 @/usr/share/gnome-shell/js/misc/fileUtils.js:51 onNextFileComplete@/usr/share/gnome-shell/js/misc/fileUtils.js:21 main@/usr/share/gnome-shell/js/extensionPrefs/main.js:276 @:1

I guess it is because my hddtemp shows strange character (you see, before the temperature). Do not know why.

/dev/sda: Samsung SSD 840 PRO Serise �: 41 C /dev/sdb: Generic Flash HS-CF: S.M.A.R.T. not available /dev/sdc: Generic Flash HS-COMBO: S.M.A.R.T. not available

Fedora 20 with all the latest updates. The extension is from git.

svu commented 10 years ago

It seems there is a bug in Fedora's hddtemp - incorrectly decoding SSD id. But that should not cause the crash of the extension settings, should it?