HEADS UP: set_rcvar() removed from rc.subr

Chris Rees crees at freebsd.org
Sat Jan 14 15:30:47 UTC 2012


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@, 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
> @@ -16,7 +16,7 @@
>  . /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


More information about the freebsd-current mailing list