From owner-freebsd-current@FreeBSD.ORG Wed May 11 19:28:41 2005 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 5562116A4CE; Wed, 11 May 2005 19:28:41 +0000 (GMT) Received: from pooker.samsco.org (pooker.samsco.org [168.103.85.57]) by mx1.FreeBSD.org (Postfix) with ESMTP id D16D243D4C; Wed, 11 May 2005 19:28:40 +0000 (GMT) (envelope-from scottl@samsco.org) Received: from [127.0.0.1] ([168.103.85.57]) (authenticated bits=0) by pooker.samsco.org (8.13.3/8.13.3) with ESMTP id j4BJX7iS042376; Wed, 11 May 2005 13:33:08 -0600 (MDT) (envelope-from scottl@samsco.org) Message-ID: <42825CC3.1060409@samsco.org> Date: Wed, 11 May 2005 13:28:03 -0600 From: Scott Long User-Agent: Mozilla/5.0 (X11; U; FreeBSD i386; en-US; rv:1.7.5) Gecko/20050218 X-Accept-Language: en-us, en MIME-Version: 1.0 To: Darren Reed References: <20050502201921.4F0387306E@freebsd-current.sentex.ca> <20050507192448.GA1401@gothmog.gr> <200505072153.54369.josemi@freebsd.jazztel.es> <20050507201142.GA9298@gothmog.gr> <427D82C4.4000201@samsco.org> <20050510154035.GB67017@hub.freebsd.org> In-Reply-To: <20050510154035.GB67017@hub.freebsd.org> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-2.8 required=3.8 tests=ALL_TRUSTED autolearn=failed version=3.0.2 X-Spam-Checker-Version: SpamAssassin 3.0.2 (2004-11-16) on pooker.samsco.org cc: Darren Reed cc: freebsd-current@freebsd.org cc: Giorgos Keramidas cc: Jose M Rodriguez Subject: Re: IPFILTER Branch [was: Re: [current tinderbox] failure on powerpc/powerpc] X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 11 May 2005 19:28:41 -0000 Darren Reed wrote: > On Sat, May 07, 2005 at 09:08:52PM -0600, Scott Long wrote: > >>Darren's latest import wasn't ever tested on FreeBSD, nor did he make >>any attempt to make it compile on FreeBSD before he imported it. > > > That's just utter bullshit. > > >>He merely dropped it in CVS and walked away. > > > As is that. I spent the best partof the next 2 to 3 days getting > it together. But I could be generous and accept that it appeared > like that but really, what you call "walking away" is what I call > "sleeping." > > Maybe you have more time in your life such that you can do commits > in hours that aren't next to when you sleep. I'm not so fortunate. > > Or maybe you just don't need to sleep. > > Or maybe you're just ignorant that people do work when they can and > in timezones that don't align well with most European/North Americans. > > >>If it wasn't for the teamwork of others, it would still be broken. > > > This time there were some parts that needed work that were outside my > experience for FreeBSD. That's unlikely to be needed again now that the > framework has changed. > > >>CVS is where work goes that is done and tested, not where undercooked >>hacks go while waiting for others to clean them up. > > > It's always tested (as in it works) before it is imported. > > The only issue I generally have is with driving CVS and making sure all > the right bits go in the right place and that it compiles cleanly. > > If I had enough "stuff" I'd rsync out the repo, do a test of everything > against a clone of the repo, test that a few times and then commit into > FreeBSD. Well, at least for the last import. But the only resource *I* > have (currently) are those in freebsd.org and my laptop that needs to be > used for a lot more things (like run FreeBSD 4 and FreeBSD 5, too.) > > Darren I completely understand that integrating IPFilter into FreeBSD is not an easy process. I also completely understand that it's impossible to do a vendor import with CVS and have the result work immediately. Vendor imports are often a process of import-then-fixup. That's fine. However, the fact that it took 3 days to do the fixup portion points to the fact that no integration work was done before hand. Before you hit the commit button, you should have already known how to fix the hard problems that you encountered. You should have already put everything in place on either local or remote machine and gotten through a buildworld. But you didn't, you assumed that you could do the import blindly and that there would be few or no problems. That cost us 3 days of the tree being broken. And that is completely unacceptable. Sun wouldn't accept their tree being broken for three days. The tools and the resources exist within FreeBSD to avoid this, there is simply no excuse to flagrantly disregard the minimum standards that we have. CVS is not a sandbox, not even the HEAD branch. If you want a sandbox then use Perforce, or at least use a projects/ branch in CVS. That's about all I'm going to say about that. Scott