svn commit: r220153 - head/etc/rc.d

Ed Maste emaste at freebsd.org
Sun Apr 3 16:16:53 UTC 2011


On Sun, Apr 03, 2011 at 02:56:50PM +0000, Bjoern A. Zeeb wrote:

> On Wed, 30 Mar 2011, Ed Maste wrote:

> > Replace ${SYSCTL_W} with ${SYSCTL} in rc.d scripts, as they are identical.
> 
>              ... for example OSX really requires the -w to alter a
> sysctl while we permit without.   I thin kthe correct fix should have
> been to add a -w to SYSCTL_W.

NetBSD also requires -w as far as I know.  However, we've deprecated the
switch -- see sysctl(8):

   The -w option has been deprecated and is silently ignored.

I don't think there's much value in using a deprecated and ignored flag
in our startup scripts.

> That way one can grep for a read vs.
> write in the startup scrpits a lot more easily, etc.

Something along the lines of egrep -ir 'sysctl.*=' can be used to find
places that sysctls are written; this has the benefit of finding the
cases where we're already not using a ${SYSCTL} or ${SYSCTL_W} variable
today.

-Ed


More information about the svn-src-all mailing list