https://bugzilla.novell.com/show_bug.cgi?id=768354
https://bugzilla.novell.com/show_bug.cgi?id=768354#c3
Jean Delvare changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|RESOLVED |REOPENED
Resolution|FIXED |
--- Comment #3 from Jean Delvare 2012-11-02 09:15:33 UTC ---
I would decline this request but apparently I am not allowed to do so.
Thanks a lot for looking into this as I was never able to make time to do it
myself. I see two issues with these changes though:
* The dependency on systemd should be optional, otherwise the package is broken
for anything older than openSUSE 12.1, including SLE11 SP2.
* The sysv init script does more than loading the kernel modules and running
"sensors -s", it also starts fancontrol if needed. So the current change
introduces a regression for users which have fancontrol set up. I don't know if
the lm_sensors unit should be modified to also start fancontrol, or if we want
to split fancontrol to a separate systemd unit. The latter seems cleaner to me.
--
Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.