Re: CVS commit: src/sbin/sysctl

[Iain Hibbert <plunky%rya-online.net@localhost>]

On Mon, 6 Apr 2009, Christos Zoulas wrote:

> In article <Pine.NEB.4.64.0904060926460.477%tx.reedmedia.net@localhost>,
> Jeremy C. Reed <reed%reedmedia.net@localhost> wrote:
> >> > Modified Files:
> >> >  src/sbin/sysctl: sysctl.8 sysctl.c
> >> >
> >> > Log Message:
> >> > Don't print an error with sysctl -w name?=value if name does not exist.
> >> > Should be pulled up to 5.0
> >>
> >> i think it's better to use something which does not have a special meaning
> >> for shells.
> >
> >Maybe a switch? (-Q)
>
> That is too wide a brush. It will stop warning on all variables in the file.

What distinguishes variables which require no warning if they do not
exist?  A switch is imo preferable because it can be turned on or off
without search/replace in the config file .. (and -qq can be used for
'really quiet' :)

On the other hand, the projected use for this is probably with -f rather
than at the command line anyway?  If so, I wouldn't see a problem with
using '?'

iain