From owner-freebsd-current Thu May 30 0:44:37 2002 Delivered-To: freebsd-current@freebsd.org Received: from pintail.mail.pas.earthlink.net (pintail.mail.pas.earthlink.net [207.217.120.122]) by hub.freebsd.org (Postfix) with ESMTP id 39E9D37B417; Thu, 30 May 2002 00:44:01 -0700 (PDT) Received: from pool0168.cvx40-bradley.dialup.earthlink.net ([216.244.42.168] helo=mindspring.com) by pintail.mail.pas.earthlink.net with esmtp (Exim 3.33 #2) id 17DKbG-0002h9-00; Thu, 30 May 2002 00:43:58 -0700 Message-ID: <3CF5D81A.7559A8D5@mindspring.com> Date: Thu, 30 May 2002 00:43:22 -0700 From: Terry Lambert X-Mailer: Mozilla 4.7 [en]C-CCK-MCD {Sony} (Win98; U) X-Accept-Language: en MIME-Version: 1.0 To: Matthew Dillon Cc: "J. Mallett" , David O'Brien , Julian Elischer , John Baldwin , FreeBSD current users Subject: Re: Seeking OK to commit KSE MIII References: <20020529124434.A2156@dragon.nuxi.com> <200205292007.g4TK71YD062671@apollo.backplane.com> <20020529152159.A82752@FreeBSD.ORG> <200205292353.g4TNrdH7063967@apollo.backplane.com> <20020529171351.A5586@FreeBSD.ORG> <200205300049.g4U0ne16064403@apollo.backplane.com> Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Sender: owner-freebsd-current@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG Matthew Dillon wrote: > But that is not what is going on here. Not by a long shot. We have > people on this list that complain over the smallest 'infraction' of the > rules, and then jump up the alleged significance of the event by > foretelling gloom and doom and the end of all things if the rules are > not followed exactly to a T. > > It MUST STOP. It is not good for the development community or the > project. Rules are what seperate us from the apes. Apes with Internet access, degrees in CS, a working knowledge of CVS, an ability to code, mailing list access, and commit bits. Oh wait. That was "the ability to grow UNIX guru beards" that seperated us from the apes. Never mind... -- I can see both sides of the argument. I've seen the "please make style commits seperately" argument used to squelch style changes, which are nearly impossible to justify on their own. I've also seen style changes go in with other changes, and the style changes just ended up being gratuitous diffs against the other BSD's, and made it harder to share code. Mostly, I dislike both points: the first is to prevent necessary change, and the second might be attributable to laziness. I've personally posted patches that inverted the logic of functions, which people have claimed were style changes. I guess they were, until you later go in and add the asserts on entry/exit lock state, etc.. -- Julian's penchant for putting squiggly braces around single statements is something I ran into when working with him at Whistle. Frankly, it doesn't bother me one way or the other. It's mostly useless (IMO), but it makes it easier to put in debugging statements that don't end up changing logic (Julian's initial argument to me; I like putting adjunct squigglies in the first column with the debugging statements themselves, personally). As gratuitous diffs go, these are incredibly minor. I think most gratuitous diffs disappear anyway, when you make "diff" ignore whitespace. Most complaints work out to be an inability on the part of the maintainer to work "diff", and whining that because they can't work the back end of the hammer, no one should use the front end of one to pound in nails, because they would be unable to pull them out later wigth their own hammers. My answer to that is "a craftsman knows his tools". Hell, if it makes things easier for Julian to put in more code, then it's OK with me. He's done the lion's share of work on getting KSE into FreeBSD so far (IMO). That's my opinion, FWIW. -- Terry To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-current" in the body of the message