Subject: Re: CVS commit: src/sys/arch
To: YAMAMOTO Takashi <yamt@mwd.biglobe.ne.jp>
From: Quentin Garnier <cube@cubidou.net>
List: source-changes
Date: 07/30/2005 21:16:59
--sIsKpy5+/8vj5Foo
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Sun, Jul 31, 2005 at 03:37:28AM +0900, YAMAMOTO Takashi wrote:
> > > Log Message:
> > > add "options VMSWAP" to non INSTALL kernels.
> >=20
> > Wouldn't it make sense to invert the option, i.e. "options VMNOSWAP"?
> >=20
> > -- thorpej
>=20
> 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.

--=20
Quentin Garnier - cube@cubidou.net - cube@NetBSD.org
"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.

--sIsKpy5+/8vj5Foo
Content-Type: application/pgp-signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.6 (NetBSD)

iQEVAwUBQuvSK9goQloHrPnoAQLZfAf/TLVYucCKNOMyj6TvhbxpBqkpa8mmcDsW
R3VzBN7IKmQezkxZUDrDd7sW1f3sJFI6vQPfzqBWzItqOv0gTVAeBbkl7q6XgFqF
vKDrpq7bEq11edtU7aBr/ilKtrEwsBDN7kbMgU34v7fPJK2Urw1Xt7LQdQhL2pUk
SUQDSQHl9xE+CfK8BJNJj0LqZ9/lfsYm3ARW+3LOrgTPTaN6L4C+arWQ8iZVmbgS
MPrcjDLy/saHad896GjKOCfDtBqTJFqjAJmj9I6M7goxm9frDxpeneCIpk0dZ3RX
vPs3fsdnqy+k/0PDM83GYR/jBffsWmhc86uEFKRljS1ULVqhyi2oMw==
=UE5U
-----END PGP SIGNATURE-----

--sIsKpy5+/8vj5Foo--