Re: HEADS UP: set_rcvar() removed from rc.subr

From: Chris Rees <crees_at_freebsd.org>
Date: Sat, 14 Jan 2012 15:30:15 +0000
On 14 January 2012 15:16, Rainer Hurling <rhurlin_at_gwdg.de> wrote:
> On 14.01.2012 10:05 (UTC+1), Doug Barton wrote:
>>
>> Howdy,
>>
>> Per discussion in freebsd-rc_at_, I have removed set_rcvar() from rc.subr.
>> The concept of set_rcvar() was nice in theory, but the forks it creates
>> are a drag on the startup process, which is especially noticeable on
>> slower systems, such as embedded ones.
>>
>> I have no plans to MFC this change, so it should only affect users who
>> are actually on 10-current. If you have scripts in /usr/local/etc/rc.d
>> (which if you have ports installed you almost certainly do) ...
>>
>> to make the change by hand, change this:
>>
>> name=foo
>> rcvar=`set_rcvar`
>>
>> to:
>>
>> name=foo
>> rcvar=foo_enable
>>
>> I didn't bump PORTREVISIONs because the change only applies to HEAD. But
>> all of the ports are updated, so if you can't figure out how to make the
>> change, just reinstall it.
>>
>>
>> Doug
>
>
> Seems that ports-mgmt/tinderbox needs an update like this:
>
> files/patch-etc__rc.d__tinderd
>
> --- etc/rc.d/tinderd.orig       2011-11-20 07:01:09.000000000 +0100
> +++ etc/rc.d/tinderd    2012-01-14 16:07:38.000000000 +0100
> _at__at_ -16,7 +16,7 _at__at_
>  . /etc/rc.subr
>
>  name="tinderd"
> -rcvar=`set_rcvar`
> +rcvar=tinderd_enable
>
>  # read settings, set default values
>  load_rc_config "${name}"
>

I'm in the process of fixing this upstream.

Chris
Received on Sat Jan 14 2012 - 14:30:46 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:40:23 UTC