[lm-sensors] Enhancement request: fancontrol error messages to

[lm-sensors] Enhancement request: fancontrol error messages to

[Charles]

Hello :-)

How about modifying the fancontrol script to write error messages to stderr?

It would be more conventional and, for example could be used to make it
less verbose -- while allowing any error messages to appear -- when run
from boot scripts  by something like:

fancontrol > /dev/null

Best

Charles

_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

Re: [lm-sensors] Enhancement request: fancontrol error messages to

[Jean Delvare]

Hi Charles,

On Fri, 02 Dec 2011 15:25:39 +0530, Charles wrote:
> Hello :-)
> 
> How about modifying the fancontrol script to write error messages to stderr?
> 
> It would be more conventional and, for example could be used to make it
> less verbose -- while allowing any error messages to appear -- when run
> from boot scripts  by something like:
> 
> fancontrol > /dev/null

I agree this makes sense. I've made the necessary changes, please give a try to:
  http://khali.linux-fr.org/devel/misc/fancontrol
Make sure you don't get a cached version, as I just updated it.

If it works OK for you, I'll commit both changes to the repository.

-- 
Jean Delvare
http://khali.linux-fr.org/wishlist.html

_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

Re: [lm-sensors] Enhancement request: fancontrol error messages to

[Charles]

On 04/12/11 20:03, Jean Delvare wrote:
>> > How about modifying the fancontrol script to write error messages to stderr?
>> > 
>> > It would be more conventional and, for example could be used to make it
>> > less verbose -- while allowing any error messages to appear -- when run
>> > from boot scripts  by something like:
>> > 
>> > fancontrol > /dev/null
> I agree this makes sense. I've made the necessary changes, please give a try to:
>   http://khali.linux-fr.org/devel/misc/fancontrol
> Make sure you don't get a cached version, as I just updated it.
> 
> If it works OK for you, I'll commit both changes to the repository.

Thanks Jean :)

It tests OK (no output) when run by

fancontrol >/dev/null &

Best

Charles


_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

Re: [lm-sensors] Enhancement request: fancontrol error messages to

[Jean Delvare]

On Sun, 04 Dec 2011 21:48:21 +0530, Charles wrote:
> On 04/12/11 20:03, Jean Delvare wrote:
> >> > How about modifying the fancontrol script to write error messages to stderr?
> >> > 
> >> > It would be more conventional and, for example could be used to make it
> >> > less verbose -- while allowing any error messages to appear -- when run
> >> > from boot scripts  by something like:
> >> > 
> >> > fancontrol > /dev/null
> > I agree this makes sense. I've made the necessary changes, please give a try to:
> >   http://khali.linux-fr.org/devel/misc/fancontrol
> > Make sure you don't get a cached version, as I just updated it.
> > 
> > If it works OK for you, I'll commit both changes to the repository.
> 
> Thanks Jean :)
> 
> It tests OK (no output) when run by
> 
> fancontrol >/dev/null &

Thanks for the report. Patches committed:
http://www.lm-sensors.org/changeset/6000
http://www.lm-sensors.org/changeset/6001

-- 
Jean Delvare
http://khali.linux-fr.org/wishlist.html

_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors