From owner-svn-src-all@FreeBSD.ORG Sun Aug 23 06:09:40 2009 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id B9603106568F for ; Sun, 23 Aug 2009 06:09:40 +0000 (UTC) (envelope-from dougb@FreeBSD.org) Received: from mail2.fluidhosting.com (mx21.fluidhosting.com [204.14.89.4]) by mx1.freebsd.org (Postfix) with ESMTP id 66B2D8FC0C for ; Sun, 23 Aug 2009 06:09:40 +0000 (UTC) Received: (qmail 6602 invoked by uid 399); 23 Aug 2009 06:09:33 -0000 Received: from localhost (HELO foreign.dougb.net) (dougb@dougbarton.us@127.0.0.1) by localhost with ESMTPAM; 23 Aug 2009 06:09:33 -0000 X-Originating-IP: 127.0.0.1 X-Sender: dougb@dougbarton.us Message-ID: <4A90DD18.2030107@FreeBSD.org> Date: Sat, 22 Aug 2009 23:09:28 -0700 From: Doug Barton Organization: http://www.FreeBSD.org/ User-Agent: Thunderbird 2.0.0.23 (X11/20090822) MIME-Version: 1.0 To: Juli Mallett References: <200908230545.n7N5jcpZ047952@svn.freebsd.org> In-Reply-To: X-Enigmail-Version: 0.95.7 OpenPGP: id=D5B2F0FB Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r196435 - head/sys/conf X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 23 Aug 2009 06:09:40 -0000 Juli Mallett wrote: > On Sat, Aug 22, 2009 at 22:45, Doug Barton wrote: >> Author: dougb >> Date: Sun Aug 23 05:45:38 2009 >> New Revision: 196435 >> URL: http://svn.freebsd.org/changeset/base/196435 >> >> Log: >> The svnversion string is only relevant when newvers.sh is called >> during the kernel build process, the other places that call the script >> do not make use of that information. So restrict execution of the >> svnversion-related code to the kernel build context. > > This seems to assume that the kernel is always checked out in a > directory called 'sys', Yes, I think that's a reasonable assumption. > Is it harmful to include the svnversion stuff for other newvers.sh > users? There was whinging that it took a "long time" to run svnversion on the whole src tree, so a very badly formed patch was applied to try and conditionalize it for only running when the kernel is being built. This change will actually restrict it to when the kernel is being built in a directory called */sys/* which will actually work for the vast majority of FreeBSD users. > Could they just set an environment variable to indicate they > want it excluded? Or could you check the environment for kernel > build-related variables? Or even set a variable when running > newvers.sh for the kernel to explicitly request svnversion inclusion? Knock yourself out, I look forward to reviewing your patches. This very simple little item has already received WAY more hackery than it really merits. The only reason I made the change I did (after swearing off touching it again on the principle of trying to avoid getting bike shed paint all over my clothes) was that the patch that was applied previously was just plain silly. FWIW, your comments would have been a lot more timely if they had been in response to my posting this patch following the discussion on the topic in -current on 8/15. Given that I have received no comments on that post at all in the 7 days since posting the patch I felt it was more than reasonable to commit it tonight. Doug -- This .signature sanitized for your protection