From owner-svn-src-user@freebsd.org Mon Mar 26 18:27:36 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 BB656F5E57B for ; Mon, 26 Mar 2018 18:27:35 +0000 (UTC) (envelope-from ian@freebsd.org) Received: from outbound1b.ore.mailhop.org (outbound1b.ore.mailhop.org [54.200.247.200]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 3D96F6B9DF for ; Mon, 26 Mar 2018 18:27:34 +0000 (UTC) (envelope-from ian@freebsd.org) X-MHO-User: 1b3bd53d-3123-11e8-bb8e-b35b57339d60 X-Report-Abuse-To: https://support.duocircle.com/support/solutions/articles/5000540958-duocircle-standard-smtp-abuse-information X-Originating-IP: 67.177.211.60 X-Mail-Handler: DuoCircle Outbound SMTP Received: from ilsoft.org (unknown [67.177.211.60]) by outbound1.ore.mailhop.org (Halon) with ESMTPSA id 1b3bd53d-3123-11e8-bb8e-b35b57339d60; Mon, 26 Mar 2018 18:25:49 +0000 (UTC) Received: from rev (rev [172.22.42.240]) by ilsoft.org (8.15.2/8.15.2) with ESMTP id w2QIQPaZ057933; Mon, 26 Mar 2018 12:26:25 -0600 (MDT) (envelope-from ian@freebsd.org) Message-ID: <1522088785.49673.25.camel@freebsd.org> Subject: Re: svn commit: r331461 - in user/markj/netdump/sys: kern netinet/netdump sys vm From: Ian Lepore To: John Baldwin , rgrimes@freebsd.org Cc: Mark Johnston , Andriy Gapon , src-committers@freebsd.org, svn-src-user@freebsd.org Date: Mon, 26 Mar 2018 12:26:25 -0600 In-Reply-To: <1621825.K6AdhHRnSW@ralph.baldwin.cx> References: <201803241540.w2OFeOcX037035@pdx.rh.CN85.dnsmgr.net> <1621825.K6AdhHRnSW@ralph.baldwin.cx> Content-Type: text/plain; charset="ISO-8859-1" X-Mailer: Evolution 3.18.5.1 FreeBSD GNOME Team Port Mime-Version: 1.0 Content-Transfer-Encoding: 8bit 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 18:27:36 -0000 On Mon, 2018-03-26 at 10:12 -0700, John Baldwin 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. > > 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. > The support for armv6 happened this way... lots of out-of-tree and project-branch work, with nothing much useful in the way of commit messages, followed by a single massive import with a commit summary of something like "import armv6 support". Now when you try to search svn history for how something came to be, all you can find out is that it was part of the incomprehensibly-huge initial commit.  You can't even figure out who to ask about something, let alone why something was done. -- Ian