From owner-svn-src-head@FreeBSD.ORG Wed Mar 10 02:40:25 2010 Return-Path: Delivered-To: svn-src-head@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 5261E106564A; Wed, 10 Mar 2010 02:40:25 +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 027738FC1E; Wed, 10 Mar 2010 02:40:24 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by harmony.bsdimp.com (8.14.3/8.14.1) with ESMTP id o2A2ZAQv006300; Tue, 9 Mar 2010 19:35:10 -0700 (MST) (envelope-from imp@bsdimp.com) Date: Tue, 09 Mar 2010 19:35:12 -0700 (MST) Message-Id: <20100309.193512.775474398290078267.imp@bsdimp.com> To: dougb@FreeBSD.org From: "M. Warner Losh" In-Reply-To: <4B97030A.1040704@FreeBSD.org> References: <4B945CAC.9020706@FreeBSD.org> <20100310015551.GA12405@dragon.NUXI.org> <4B97030A.1040704@FreeBSD.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: svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org, obrien@FreeBSD.org Subject: Re: svn commit: r204824 - head/sys/conf X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 10 Mar 2010 02:40:25 -0000 In message: <4B97030A.1040704@FreeBSD.org> Doug Barton writes: : On 03/09/10 17:55, David O'Brien wrote: : > On Sun, Mar 07, 2010 at 06:10:52PM -0800, Doug Barton wrote: : >> On 03/07/10 17:01, David O'Brien wrote: : >>> +*/sys/*|*/compile/*) : >> : >> Please revert your previous change, and feel free to add this one instead. : >> : >> I'm a little aggravated because this question of "what if the directory : >> isn't named */sys/*?" was already addressed when I made the change, and : >> the general agreement was that !sys was an edge case that wasn't worth : >> worrying about. : > : > I probably would have if I had been able to update my main system to a : > kernel newer than Nov 9th. Every time I've tried HEAD is toxic[*]. : > Tried again this morning - "no route to 127.0.0.1". : : I'm not sure what the connection is there. But FWIW Qing posted the fix : for that: http://people.freebsd.org/~qingli/route.h.diff. Not sure why : he hasn't committed it yet. The fix works for me. : : >> I'm even more aggravated that your change doesn't work for the common case. : > : > Sorry, I build all my kernels the traditional way. I was agrivated that : > your change didn't work with the kernel build method with 15 years : > history... : : The "traditional" way hasn't been the recommended method of kernel : building for years now. : : In any case, at least one user has complained on the list that not : having the svn revision in uname has caused them an actual problem, so I : backed out your change today. If you'd like to submit a patch for review : (-current or -hackers are good choices) that ADDS support for : */compile/* I have no objection, and as long as it doesn't cause any : problems for people whose sources are in */sys/* I think that'd be : great, and will do whatever I can to help. I'd recommend something similar to the following change: Index: newvers.sh =================================================================== --- newvers.sh (revision 204938) +++ newvers.sh (working copy) @@ -38,13 +38,14 @@ fi RELEASE="${REVISION}-${BRANCH}" VERSION="${TYPE} ${RELEASE}" +S=sys if [ "X${PARAMFILE}" != "X" ]; then RELDATE=$(awk '/__FreeBSD_version.*propagated to newvers/ {print $3}' \ ${PARAMFILE}) else RELDATE=$(awk '/__FreeBSD_version.*propagated to newvers/ {print $3}' \ - $(dirname $0)/../sys/param.h) + $(dirname $0)/../$S/param.h) fi @@ -88,15 +89,15 @@ i=`${MAKE:-make} -V KERN_IDENT` case "$d" in -*/sys/*) +*/$S/*) SRCDIR=${d##*obj} if [ -n "$MACHINE" ]; then SRCDIR=${SRCDIR##/$MACHINE} fi - SRCDIR=${SRCDIR%%/sys/*} + SRCDIR=${SRCDIR%%/$S/*} for dir in /bin /usr/bin /usr/local/bin; do - if [ -d "${SRCDIR}/sys/.svn" -a -x "${dir}/svnversion" ] ; then + if [ -d "${SRCDIR}/$S/.svn" -a -x "${dir}/svnversion" ] ; then svnversion=${dir}/svnversion break fi @@ -107,7 +108,7 @@ done if [ -n "$svnversion" ] ; then - svn=" r`cd ${SRCDIR}/sys && $svnversion`" + svn=" r`cd ${SRCDIR}/$S && $svnversion`" fi if [ -n "$git_cmd" ] ; then git=`$git_cmd rev-parse --verify --short HEAD 2>/dev/null` which would allow people that move sys to somewhere else to change one line and still have everything work. They are going to have to make changes anyway to support that, since it isn't supported by FreeBSD out of the box. Warner