From 3d00d349c6eeac3f3c33444a0c867b1927182ad3 Mon Sep 17 00:00:00 2001 From: "Sean E. Russell" Date: Mon, 30 Nov 2020 10:35:24 -0600 Subject: [PATCH] Fixes #130. Any warning or error from gopsutil was preventing any temps from being displayed, even if some had useful data. --- devices/cpu_linux.go | 1 + devices/temp_linux.go | 8 ++++++-- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/devices/cpu_linux.go b/devices/cpu_linux.go index 4e836d5..2368218 100644 --- a/devices/cpu_linux.go +++ b/devices/cpu_linux.go @@ -2,6 +2,7 @@ package devices +// TODO gopsutil is at v3, and we're using v2. See if v3 is released and upgrade if so. import "github.com/shirou/gopsutil/cpu" func CpuCount() (int, error) { diff --git a/devices/temp_linux.go b/devices/temp_linux.go index 0bd480f..b738002 100644 --- a/devices/temp_linux.go +++ b/devices/temp_linux.go @@ -3,6 +3,7 @@ package devices import ( + "log" "strings" "github.com/shirou/gopsutil/host" @@ -15,8 +16,11 @@ func devs() []string { } sensors, err := host.SensorsTemperatures() if err != nil { - // FIXME report the error - return []string{} + log.Printf("gopsutil reports %s", err) + if len(sensors) == 0 { + log.Printf("no temperature sensors returned") + return []string{} + } } rv := make([]string, 0, len(sensors)) for _, sensor := range sensors {