Source-Changes archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/sys/arch



On Sun, Jul 31, 2005 at 03:37:28AM +0900, YAMAMOTO Takashi wrote:
> > > Log Message:
> > > add "options VMSWAP" to non INSTALL kernels.
> > 
> > Wouldn't it make sense to invert the option, i.e. "options VMNOSWAP"?
> > 
> > -- thorpej
> 
> i don't think so.
> negative options are evil and confusing IMO.

Hmm.  Don't expect the average user to go through options(4) (you added
that to options(4) already, right?) to understand the meaning of
"VMSWAP" which is not, believe it or not, completely obvious at first
sight.

And note how it is not "NOVMSWAP", but rather "VMNOSWAP" what Jason
proposed.

But anyway, as a general rule, it is bad to require adding a line to
every kernels to get the previous working state.  When you do that,
you'd better put warnings in capital letters in UPDATING and a "HEADS
UP" in -current so that people will update their very own config files.

Confusing, indeed.

-- 
Quentin Garnier - cube%cubidou.net@localhost - cube%NetBSD.org@localhost
"When I find the controls, I'll go where I like, I'll know where I want
to be, but maybe for now I'll stay right here on a silent sea."
KT Tunstall, Silent Sea, Eye to the Telescope, 2004.

Attachment: pgpcS6U5pLegK.pgp
Description: PGP signature



Home | Main Index | Thread Index | Old Index