From owner-freebsd-arch Fri Mar 15 7:35:38 2002 Delivered-To: freebsd-arch@freebsd.org Received: from mail12.speakeasy.net (mail12.speakeasy.net [216.254.0.212]) by hub.freebsd.org (Postfix) with ESMTP id 938E137B449 for ; Fri, 15 Mar 2002 07:34:31 -0800 (PST) Received: (qmail 23947 invoked from network); 15 Mar 2002 15:34:30 -0000 Received: from unknown (HELO server.baldwin.cx) ([216.27.160.63]) (envelope-sender ) by mail12.speakeasy.net (qmail-ldap-1.03) with DES-CBC3-SHA encrypted SMTP for ; 15 Mar 2002 15:34:30 -0000 Received: from laptop.baldwin.cx (gw1.twc.weather.com [216.133.140.1]) by server.baldwin.cx (8.11.6/8.11.6) with ESMTP id g2FFYuv39818; Fri, 15 Mar 2002 10:34:57 -0500 (EST) (envelope-from jhb@FreeBSD.org) Message-ID: X-Mailer: XFMail 1.5.2 on FreeBSD X-Priority: 3 (Normal) Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 8bit MIME-Version: 1.0 In-Reply-To: Date: Fri, 15 Mar 2002 10:34:32 -0500 (EST) From: John Baldwin To: Dag-Erling Smorgrav Subject: Re: Unmoronify CVS Cc: arch@FreeBSD.ORG, Peter Wemm Sender: owner-freebsd-arch@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On 15-Mar-2002 Dag-Erling Smorgrav wrote: > Peter Wemm writes: >> But that would be slow if it had to recursively search for RELENG_4 each >> time you did a 'cvs up -r RELENG_4' in /usr/src, so they added a cache. I >> do not recall whether it aborts the search on the first match, or if it >> searches everything. I believe it is the latter (ie: if there is no >> val-tags match, it will search the entire tree). > > Yep, even if it finds the tag in the very first file. > >> The problem with removing val-tags is that cvs will now search *always* >> before doing an update. Effectively, you will be condeming every end-user >> to an exhaustive search each time they do a cvs update -r RELENG_4 > > No - my patch disables the entire tag-checking code. > >> If my memory serves correctly, the reason why the tag search is exhaustive >> rather than stops at the first match, is because it uses common scanning >> code for all consumers. > > Correct. > >> Looking over your patch, it seems as though it skips the recursive scan. >> Is this correct? I could live with that, but if it going to do the >> recursive check each time, then no way :-). It looks like the worst >> case is that if you accidently do this: cd /usr/src; cvs up -r RELNG_4 >> then you'll end up with an empty tree, right? > > Any sensible person would discover his mistake and abort the update > long before that, and cvs does not remove modified files, so no real > harm is done. Not if you do a cvs up -rFOO and then go to sleep only to wake up in the morning and now need to cvs up a whole big tree over your 28.8k dialup link. :) Remember, the cvs binary in the tree is used by more than just FreeBSD developers doing FreeBSD development. Add a command line option or config file option and submit it to the cvshome folks. -- John Baldwin <>< http://www.FreeBSD.org/~jhb/ "Power Users Use the Power to Serve!" - http://www.FreeBSD.org/ To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-arch" in the body of the message