From owner-cvs-all Tue Jul 17 15:28:36 2001 Delivered-To: cvs-all@freebsd.org Received: from winston.freebsd.org (adsl-64-173-15-98.dsl.sntc01.pacbell.net [64.173.15.98]) by hub.freebsd.org (Postfix) with ESMTP id 2C24E37B406; Tue, 17 Jul 2001 15:28:26 -0700 (PDT) (envelope-from jkh@freebsd.org) Received: from localhost (jkh@localhost [127.0.0.1]) by winston.freebsd.org (8.11.4/8.11.4) with ESMTP id f6HMS5t66190; Tue, 17 Jul 2001 15:28:05 -0700 (PDT) (envelope-from jkh@freebsd.org) To: obrien@freebsd.org Cc: drosih@rpi.edu, bde@zeta.org.au, jhb@freebsd.org, dd@freebsd.org, cvs-committers@freebsd.org, cvs-all@freebsd.org, wollman@khavrinen.lcs.mit.edu Subject: Re: cvs commit: src/usr.sbin/zic private.h scheck.c zic.c src/us In-Reply-To: <20010717102707.C74750@dragon.nuxi.com> References: <20010717102707.C74750@dragon.nuxi.com> X-Mailer: Mew version 1.94.1 on Emacs 20.7 / Mule 4.0 (HANANOEN) Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Message-Id: <20010717152805Y.jkh@freebsd.org> Date: Tue, 17 Jul 2001 15:28:05 -0700 From: Jordan Hubbard X-Dispatcher: imput version 20000228(IM140) Lines: 45 Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG From: "David O'Brien" Subject: Re: cvs commit: src/usr.sbin/zic private.h scheck.c zic.c src/us Date: Tue, 17 Jul 2001 10:27:07 -0700 > On Tue, Jul 17, 2001 at 12:50:39PM -0400, Garance A Drosihn wrote: > > Yes, I see the bit about "merged changes from vendor branch", > > but I suspect that would not wave any red flag at me if I > > were just looking to modify this source. > > The vendor tag of "ADO: 1.1.1" should also clue you in. Actually, since I've been looking at our published guidelines rather carefully as of late, I think that John has a very valid point and Garrett does not. From the committer's guide: Don't commit to anything under the: src/contrib src/crypto and src/sys/contrib trees without explicit approval from the respective maintainer(s). The trees mentioned above are for contributed software usually imported onto a vendor branch. Committing something there, even if it doesn't take the file off the vendor branch, may cause unnecessary headaches for those responsible for maintaining that particular piece of software. Thus, unless you have explicit approval from the maintainer (or you are the maintainer), do not commit there! It doesn't say that only "big" code goes here nor is it documented ANYWHERE that one should look around for special tags before committing. In fact, the inference is very strong that those explicit locations are reserved for this kind of thing and the rest of the tree is not assumed to be vendor branched. We have a lot of new committers coming into FreeBSD with more being added every day. If we can't document our own processes or claim that people should just somehow magically be familiar with "established precedent" then we're doing them a real disservice and making our own claims of being an organized development process somewhat hypocritical at best. The case of "zic" is a clear case of "personal preference over policy" and not something really open to debate, not that that's ever stopped us from debating things before.. A little consistency would be nice here folks - it's frankly bizarre to see a discussion about how much the rules matter and another about how they don't going on at the same time. - Jordan To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message