From owner-freebsd-current@FreeBSD.ORG Thu Aug 8 15:59:29 2013 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTP id DCF54E0F; Thu, 8 Aug 2013 15:59:28 +0000 (UTC) (envelope-from gjb@FreeBSD.org) Received: from mail0.glenbarber.us (mail0.glenbarber.us [208.86.227.67]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id 96C0E2E9B; Thu, 8 Aug 2013 15:59:28 +0000 (UTC) Received: from glenbarber.us (unknown [IPv6:2001:470:8:120e:1:1:c57c:729]) (using TLSv1 with cipher ECDHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) (Authenticated sender: gjb) by mail0.glenbarber.us (Postfix) with ESMTPSA id 48D00A7DD; Thu, 8 Aug 2013 15:59:26 +0000 (UTC) DKIM-Filter: OpenDKIM Filter v2.8.3 mail0.glenbarber.us 48D00A7DD Authentication-Results: mail0.glenbarber.us; dkim=none reason="no signature"; dkim-adsp=none Date: Thu, 8 Aug 2013 11:59:24 -0400 From: Glen Barber To: John Baldwin Subject: Re: svn error during 'make buildkernel'? Message-ID: <20130808155924.GA29016@glenbarber.us> References: <20130803210348.GA715@troutmask.apl.washington.edu> <20130806181107.GR34979@over-yonder.net> <20130806183054.GB2190@glenbarber.us> <201308081114.05978.jhb@freebsd.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="LQksG6bCIzRHxTLp" Content-Disposition: inline In-Reply-To: <201308081114.05978.jhb@freebsd.org> X-Operating-System: FreeBSD 10.0-CURRENT amd64 User-Agent: Mutt/1.5.21 (2010-09-15) Cc: freebsd-current@freebsd.org, Steve Kargl , Peter Wemm , "Matthew D. Fuller" X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.14 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: Thu, 08 Aug 2013 15:59:29 -0000 --LQksG6bCIzRHxTLp Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Aug 08, 2013 at 11:14:05AM -0400, John Baldwin wrote: > On Tuesday, August 06, 2013 2:30:54 pm Glen Barber wrote: > > On Tue, Aug 06, 2013 at 01:11:07PM -0500, Matthew D. Fuller wrote: > > > On Sun, Aug 04, 2013 at 08:55:30PM -0400 I heard the voice of > > > Glen Barber, and lo! it spake thus: > > > >=20 > > > > The error generated is non-fatal, and once I receive response on a > > > > proposed patch, will be suppressed if the svn version used to check > > > > out the tree is not compatible with that used to check the version > > > > of the tree with the kernel build. > > >=20 > > > But not try the ports svn as well? I mean, as breakage goes, it's not > > > even in the top 100; I'd _much_ rather have a kernel that I have to > > > guess the revision of but boots, than one properly recorded that > > > doesn't. But it's still unpleasant, and is one of those things you > > > probably won't notice missing until suddenly you need it. > > >=20 > > > And this isn't just a presentism. Sure, right _now_ devel/subversion > > > and base svnlite get along, but what happens when ports moves to 1.9 > > > which changes the WT format? Even if -CURRENT src gets upgraded > > > simultaneously[0], the same surely can't be said of every back branch. > > >=20 > > > I realize this is all still a WIP, and please don't read any anger > > > into my words. But this _has_ been something I've found a little > > > worrisome since the original import/newvers change. Heck, newvers can > > > show me version info if I'm getting my source tree from git or p4, but > > > can't handle ports svn? By the time this works its way into a stable > > > branch, I really think it should either handle svnversion from ports > > > as well, or come with a big bright flashing warning that using svn > > > from anything but base svnlite for /usr/src is a degraded experience. > > >=20 > > >=20 > > > [0] Which still wouldn't really fix things, since > > > /usr/bin/svnliteversion is arbitrarily old, not up to date with > > > the source tree. > > >=20 > >=20 > > I have this on my todo list, but right now I have bigger things to deal > > with. As soon as I can, I will fix the logic. Right now, it is not "as > > easy as checking which svn works", because the more I look at the logic > > for newvers.sh, the more I dislike how it all works. >=20 > BTW, I was totally surprised by this recent error on my laptop which still > has 1.7 installed. I don't rebuild ports all the time because it's a PIT= A. > I think the fact that svnliteversion is used in preference to svnversion > is a huge POLA violation and completely agree with Steve on this one. > It shouldn't be that hard to just check $? and fallback to svnliteversion > if svnversion fails. I have much more complex hacks in place at work whe= re > we have active 1.6, 1.7, and 1.8 clients. :( >=20 Fixed in r254094. Glen --LQksG6bCIzRHxTLp Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.20 (FreeBSD) iQEcBAEBCAAGBQJSA8BcAAoJEFJPDDeguUajOVoH/iQwaIVhk5nqRe5Ogve5gUgy PuxACYXBzo7yEiTSE60yI0UGYeOHM3MgvP7GReMc3/O9QVsy6snq+OOsI/8RWuz3 TTfZ+dt5865oYw7+ZcT+g5J/lVJFd1hXlvbSiX92br1zXqcFdL4dhRdp+vljAXi4 WoL19bpzhbtBb4uWRtPG2c5qM3eSrnUuf8CFl9Z78l+g7tmdi3OkcVWaJQ1/EUtg 6PzBCJ4UG5ow/hsU2kOIyVbvr2bEgT/ydvqoqsdwiCN9xV7L8z9VGRw2AA6Fq44M oDgrbpBqmjW28pvLRqm9NUVUvHErYZQG32IY2GwQWukCpK8sK2pHHSNtEJYonLQ= =FdTZ -----END PGP SIGNATURE----- --LQksG6bCIzRHxTLp--