From owner-svn-src-user@freebsd.org Mon Mar 26 21:54:06 2018 Return-Path: Delivered-To: svn-src-user@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 375EAF6EB31 for ; Mon, 26 Mar 2018 21:54:06 +0000 (UTC) (envelope-from jhb@freebsd.org) Received: from mail.baldwin.cx (bigwig.baldwin.cx [IPv6:2001:470:1f11:75::1]) (using TLSv1 with cipher DHE-RSA-CAMELLIA256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id D68B076166; Mon, 26 Mar 2018 21:54:05 +0000 (UTC) (envelope-from jhb@freebsd.org) Received: from ralph.baldwin.cx (ralph.baldwin.cx [66.234.199.215]) by mail.baldwin.cx (Postfix) with ESMTPSA id 11B2110A7DB; Mon, 26 Mar 2018 17:54:05 -0400 (EDT) From: John Baldwin To: rgrimes@freebsd.org Cc: Mark Johnston , Andriy Gapon , src-committers@freebsd.org, svn-src-user@freebsd.org Subject: Re: svn commit: r331461 - in user/markj/netdump/sys: kern netinet/netdump sys vm Date: Mon, 26 Mar 2018 11:58:51 -0700 Message-ID: <126863891.cbjKKnxzry@ralph.baldwin.cx> User-Agent: KMail/4.14.10 (FreeBSD/11.1-STABLE; KDE/4.14.30; amd64; ; ) In-Reply-To: <201803261813.w2QIDX15048333@pdx.rh.CN85.dnsmgr.net> References: <201803261813.w2QIDX15048333@pdx.rh.CN85.dnsmgr.net> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.4.3 (mail.baldwin.cx); Mon, 26 Mar 2018 17:54:05 -0400 (EDT) X-Virus-Scanned: clamav-milter 0.99.2 at mail.baldwin.cx X-Virus-Status: Clean X-BeenThere: svn-src-user@freebsd.org X-Mailman-Version: 2.1.25 Precedence: list List-Id: "SVN commit messages for the experimental " user" src tree" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 26 Mar 2018 21:54:06 -0000 On Monday, March 26, 2018 11:13:33 AM Rodney W. Grimes wrote: > > On Saturday, March 24, 2018 08:40:24 AM Rodney W. Grimes wrote: > > > > On Sat, Mar 24, 2018 at 02:17:02PM +0200, Andriy Gapon wrote: > > > > > On 24/03/2018 04:46, Rodney W. Grimes wrote: > > > > > > I know this is on a private branch, but when/if it > > > > > > is merged this becomes part of the main line. > > > > > > > > > > Not with svn, I think. > > > > > At least, the way we use it. > > > > > > > > Indeed, I have no intention to merge the branch directly. I'm using an > > > > svn branch so that it's marginally easier for others to test. > > > > > > None the less as stated in: > > > https://svnweb.freebsd.org/base/projects/GUIDELINES.txt?view=markup > > > > > > 12 General guidelines: > > > 13 > > > 14 * Should be relevant to FreeBSD. > > > 15 * Should be at least conceivably of interest to somebody else. > > > 16 * Should be in a format that is suitable to merge into the base tree. > > > 17 * Should be something that is worth people's time to read commit mail for. > > > 18 * Write decent commit messages! > > > > > > Thanks, > > > > We generally don't do that for user, etc. branches. Merging from a > > projects/user branch into head in svn is often a disaster due to svn's > > limitations, so normally a projects/user branch is treated as a work area > > and the resulting diff is then hand-applied to head with a suitable commit > > message that describes the entire change. This is similar to using something > > like 'git rebase' to rewrite history and compress a long tail of changes > > down to a small number of commits prior to merging to head. > > I was quoting from a document that is specifically addressing "user and > project" branches. It seems we have a conflict of opionion on this. There is less consensus on this than that document implies. Actual uses are more varied. > > You generally don't see these work branches in svn as most developers do them > > outside of svn in git, p4, hg, etc. due to svn's limitations. > > > > For things that live permanently in user/projects (e.g. the code for core > > elections or the patches for freebsd-update), we do want standard commit > > messages. However, I don't think we want to impose that on WIP branches > > that are later compressed down before merging. > > Then why bother mailing them to @committers and having us all read > through them. I am sure that was Peters intent when he wrote this > guideline. > > And as far as I am aware all things in user/projects are permanent, > and globally mirrored. Yes, they are. I don't do any of my work branches in svn personally, but I also don't generally have to coordinate with other developers for most of my branches. -- John Baldwin