From owner-svn-src-head@FreeBSD.ORG Wed Dec 28 23:00:58 2011 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 7FC1A1065672; Wed, 28 Dec 2011 23:00:58 +0000 (UTC) (envelope-from uqs@spoerlein.net) Received: from acme.spoerlein.net (acme.spoerlein.net [IPv6:2a01:4f8:131:23c2::1]) by mx1.freebsd.org (Postfix) with ESMTP id 2F0888FC15; Wed, 28 Dec 2011 23:00:58 +0000 (UTC) Received: from localhost (acme.spoerlein.net [IPv6:2a01:4f8:131:23c2::1]) by acme.spoerlein.net (8.14.4/8.14.4) with ESMTP id pBSN0vSw024858 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES128-SHA bits=128 verify=NO); Thu, 29 Dec 2011 00:00:57 +0100 (CET) (envelope-from uqs@spoerlein.net) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=spoerlein.net; s=dkim200908; t=1325113257; bh=MFrYJH7c00ptMOOxhZvLmydfF5Zs1v997SN8UvVCooE=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Transfer-Encoding:In-Reply-To; b=O4K5AVBRvfpRzGZJvOuiQAY3J77Op5gK2XkbcEFk/oDh7GyTuPNxFniQZ12hPpo3j su5dCAaaECVk38l0pQ6U7UCPSfAOE7UhelLAyIyUjFBU+r/RwVmxyNsLdgEHuPTt9/ E+8ee+HfPeW1Aaw1yOM1n4bdk76ExoA1+KibDezA= Date: Thu, 29 Dec 2011 00:00:57 +0100 From: Ulrich =?utf-8?B?U3DDtnJsZWlu?= To: Doug Barton Message-ID: <20111228230056.GH83814@acme.spoerlein.net> Mail-Followup-To: Ulrich =?utf-8?B?U3DDtnJsZWlu?= , Doug Barton , Xin LI , src-committers@FreeBSD.org, svn-src-all@FreeBSD.org, svn-src-head@FreeBSD.org References: <201112260907.pBQ979X4098221@svn.freebsd.org> <4EF83AF4.9010108@FreeBSD.org> <20111228154049.GD83814@acme.spoerlein.net> <4EFB9386.8060909@FreeBSD.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <4EFB9386.8060909@FreeBSD.org> User-Agent: Mutt/1.5.21 (2010-09-15) Cc: svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org, Xin LI Subject: Re: svn commit: r228896 - head/contrib/netcat 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, 28 Dec 2011 23:00:58 -0000 On Wed, 2011-12-28 at 14:09:10 -0800, Doug Barton wrote: > On 12/28/2011 07:40, Ulrich Spörlein wrote: > > On Mon, 2011-12-26 at 01:14:28 -0800, Doug Barton wrote: > >> On 12/26/2011 01:07, Xin LI wrote: > >>> Author: delphij > >>> Date: Mon Dec 26 09:07:08 2011 > >>> New Revision: 228896 > >>> URL: http://svn.freebsd.org/changeset/base/228896 > >>> > >>> Log: > >>> Merge from OpenBSD 5.0 (this is a dummy change, the vendor change does not > >>> apply to us). > >> > >> When I'm importing stat(1) stuff from Net/OpenBSD I don't do this. I > >> will however comment in the commit log for the next substantive change, > >> "Skipped update N.NN because the change was not relevant to us," or > >> words to that effect. > >> > >> I'm not suggesting that my way of doing this is perfect, or cannot be > >> improved. I would suggest however that this change was needless churn. > > > > I think it was the right thing to do. It's better to have one person > > (Xin LI) figure out if the change is needed or a no-op and do the > > upgrade to match our version to upstream's version, than to have a > > discrepancy between the two and cause half a dozen developers that > > stumble upon that difference to scratch their heads and spend time in > > figuring out if we should import the change. > > Fair enough. Having thought more about this my only remaining concern is > that by slipping the tag we're misrepresenting the status quo since > what's in our tree is not really that version of the file. Perhaps a > comment to the effect of "purposely skipping version 1.23 because ..." > would be better? I always look at the CVS Ids from other projects as some kind of guideline or high water mark as to when someone has last looked at all the upstream changes and possibly brought over all the changes that apply. I mean the files are never guaranteed to be the *exact* revision 1.23 from OpenBSD, due to FreeBSD specific changes. Should every committer changing a file look for CVS Ids of other projects and then remove them as the file is being altered and no longer matches upstream bit for bit? That surely is impractical, of course no one would propose such a scheme. Adding that information to the commit message is prudent, however. In the end, whoever gets shit done is getting shit done :) Cheers, Uli