MisterGuinness / gnome-shell-extension-sensors

Gnome shell extension: Shows CPU temperature, HDD temperature, voltage and fan RPM
0 stars 0 forks source link

Fedora 35: Errors Reported by lm_sensors are Ignored #66

Closed MisterGuinness closed 2 years ago

MisterGuinness commented 2 years ago

While working on Issue #63 I noticed the sensor menu disappeared when the sensors command output anything to stderr.

Actually the stderr stream was not read at all, which somehow prevented the stdout from being read, and the lack of output caused the menu to panic and display an error instead (as if lm_sensors was not installed or sensors-detect had not been run).

I think it better to read stderr for the commands, and if present log it to the system log and include something in the menu to indicate there was an error, but still continue to display whatever sensor data is available.

MisterGuinness commented 2 years ago

image