TamtamHero / fw-fanctrl

A simple systemd service to better control Framework Laptop's fan(s)
BSD 3-Clause "New" or "Revised" License
178 stars 33 forks source link

Return "sensors -j" to null #16

Closed Originalme closed 11 months ago

Originalme commented 1 year ago

Modified original sensors -j call to output all to /dev/null. This prevents temp errors when something cannot be read. i.e. when Wi-Fi is disabled, this will cause an error to log to journal that "temp1_input" cannot be read for device.

TamtamHero commented 11 months ago

Thanks !

TamtamHero commented 11 months ago

Oh god, I should have tried that before merging :facepalm: Of course it should be sensors -j 2> /dev/null otherwise there's nothing to parse

Fixed with https://github.com/TamtamHero/fw-fanctrl/commit/5721a13f03e6cbff1d5c7cfa0923b5d1863f44a9

zquestz commented 11 months ago

That would be why I commented. =)