From owner-freebsd-current@FreeBSD.ORG Mon Mar 15 14:51:32 2010 Return-Path: Delivered-To: freebsd-current@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 86C641065673; Mon, 15 Mar 2010 14:51:32 +0000 (UTC) (envelope-from imp@bsdimp.com) Received: from harmony.bsdimp.com (bsdimp.com [199.45.160.85]) by mx1.freebsd.org (Postfix) with ESMTP id DFE778FC1A; Mon, 15 Mar 2010 14:51:31 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by harmony.bsdimp.com (8.14.3/8.14.1) with ESMTP id o2FEiEns008834; Mon, 15 Mar 2010 08:44:14 -0600 (MDT) (envelope-from imp@bsdimp.com) Date: Mon, 15 Mar 2010 08:44:26 -0600 (MDT) Message-Id: <20100315.084426.824931270219314740.imp@bsdimp.com> To: obrien@FreeBSD.org From: "M. Warner Losh" In-Reply-To: <20100315142806.GA5786@dragon.NUXI.org> References: <20100312171206.GA31761@dragon.NUXI.org> <20100313.211303.585238797224891349.imp@bsdimp.com> <20100315142806.GA5786@dragon.NUXI.org> X-Mailer: Mew version 6.3 on Emacs 22.3 / Mule 5.0 (SAKAKI) Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: freebsd-current@FreeBSD.org Subject: Re: [PATCH] newvers.sh X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.5 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: Mon, 15 Mar 2010 14:51:32 -0000 In message: <20100315142806.GA5786@dragon.NUXI.org> "David O'Brien" writes: : On Sat, Mar 13, 2010 at 09:13:03PM -0700, M. Warner Losh wrote: : > In message: <20100312171206.GA31761@dragon.NUXI.org> : > "David O'Brien" writes: : > : * Simplify SRCDIR calculation by directly finding the kernel sources : > : based directly on one of them. : > : Reviewed by: dhw : > : This change does not increase the kernel build time. It also continues : > : to restrict the revision to just the kernel sources, and not the whole : > : tree. : > : Timing tests by: dhw : > : > : > : > I have a better simplification, I think, that works for me for each of : > the tests that I've done (both traditional and buildkernel builds). : : I also have other simplifications. But want to do this in stages, : least there be some corner case or misunderstanding by someone. Normally this is a sensible process. However, my patches change how we guess where sys is, and as such, I think we should go directly there and work out the corner cases and such there. : > Can you confirm this works for you and also comment on the change : > itself? It is a bigger change, but results in a simpler (I think) : > newvers.sh. : > Comments? : : I would prefer to do this in stages so things aren't misunderstood. : This patch is one of the key parts that shows I am not making the : 'svnversion' run take longer, and that it does not cover more of : the tree than before. That's exactly what my patch does by passing in SYSDIR directly. Why guess and then replace it with a different guess. Why not go directly to the best guess immediately? We know it is always right, because we know it is what the kernel is using to build itself. And we know that's the directory we want to use for the svn command. Warner