Date: Mon, 14 Jul 2003 15:00:09 -0400 From: Garance A Drosihn <drosih@rpi.edu> To: Bruce Evans <bde@zeta.org.au>, "David O'Brien" <obrien@FreeBSD.org> Cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src Makefile.inc1 src/rescue/rescue Makefile Message-ID: <p05210660bb38acc2a751@[128.113.24.47]> In-Reply-To: <20030715035725.R8481@gamplex.bde.org> References: <200307111657.h6BGvhaX047506@repoman.freebsd.org> <20030714003736.A38298@hub.freebsd.org> <20030715035725.R8481@gamplex.bde.org>
next in thread | previous in thread | raw e-mail | index | archive | help
At 4:03 AM +1000 7/15/03, Bruce Evans wrote: >On Mon, 14 Jul 2003, David O'Brien wrote: > >> On Fri, Jul 11, 2003 at 09:57:43AM -0700, Gordon Tetlow wrote: >> > gordon 2003/07/11 09:57:43 PDT >> > >> > FreeBSD src repository >> > >> > Modified files: >> > . Makefile.inc1 >> > rescue/rescue Makefile >> > Log: > > > Turn rescue back on, conditional to NORESCUE. We seem to > > > be split on using underscores or not, ... > > > > We aren't split -- older options have no underscore, new ones > > do (unless tightly related to older options). Please change > > it to NO_RESUCE. > >We are split. Some of us don't like the underscores. Please >change it only after you change the English word "nothing" to >"no_thing" both in spelling an pronunciation. "norescue" is pronounced "no rescue" and not "nor escue". Personally I think it's much better to have 'no_blah' for options, for those cases where 'noblah' is a valid word that does not mean 'no blah', such as "NO_ISE" being obviously different than "NOISE". I picked up this line of thinking from the author of ruby, who notes that for people who are not-native to English, it is much easier to deal with 'word_word' than 'wordword'. Just my 2 cents. I'd vote for the 'NO_' form. -- Garance Alistair Drosehn = gad@gilead.netel.rpi.edu Senior Systems Programmer or gad@freebsd.org Rensselaer Polytechnic Institute or drosih@rpi.edu
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?p05210660bb38acc2a751>