On Fri, Mar 24, 2017 at 03:43:09AM -0000, Andrew Toskin wrote:
I've used both Freon and lm_sensors without blowing up my
computer,
and it since `sensors-detect` is mandatory for Freon to work, it seems
like it should be included in a %post scriptlet. But I don't want to
damage unsuspecting users' hardware... 😕
I've never suffered any damage from sensors-detect (/me crosses
fingers), but I have seen it cause kernel panics even on recent hardware
(particularly when probing the ISA bus on a 64-bit ARM system which does
not have an ISA bus), so running sensors-detect from an rpm %post
scriptlet is probably a bad idea. If it does cause a kernel panic, it's
going to cause a user panic, and it'll probably corrupt the rpm/yum/dnf
databases since it crashed in the middle of a transaction.
Can Freon check if sensors-detect has been run before, and if not, pop
up a dialog box asking to run it and warn about the risks?
--
Jeff Bastian