From owner-svn-src-user@freebsd.org Mon Mar 26 23:21:07 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 E35D4F515D7 for ; Mon, 26 Mar 2018 23:21:06 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: from mail-io0-x22b.google.com (mail-io0-x22b.google.com [IPv6:2607:f8b0:4001:c06::22b]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G2" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 74C607C51D for ; Mon, 26 Mar 2018 23:21:06 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: by mail-io0-x22b.google.com with SMTP id q80so14386111ioi.13 for ; Mon, 26 Mar 2018 16:21:06 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bsdimp-com.20150623.gappssmtp.com; s=20150623; h=mime-version:sender:in-reply-to:references:from:date:message-id :subject:to:cc; bh=hfPx+cH/xFhVCeMTzQGiEQsgMkVw3bGgITLn68dajy0=; b=Esjr3799sYsvTuhB4uJ0Cv5QUv1PKDMDzDhyaMjBvtV6AMEdEE5CjqUq1w/CVUCkis 06ywMHrLwdsIn512SjZ0wuvDQOb45jhxZD+AhRNaSRRnHKWlv8lwUx4Y64CIZ7Cj7lx5 DGmqSDlXNuKPon52Lq83u0gGI8883LsQne0EVF4yykSpMEiNaqcbwwMQHz2FYTFTG5fp J7VVRNFzF0P/Wto29lZa5dHHJwimoDpwwtysN8+tA4pjqJ+fJfi0GxQFriSjQ51o4ExT RFjtB5yQJcdKM1PKBDtPhOdTxgskq2bGSykpMmD6Anp5vJ5xSlHiw92WezPcTg78h7Cv n2dQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:sender:in-reply-to:references:from :date:message-id:subject:to:cc; bh=hfPx+cH/xFhVCeMTzQGiEQsgMkVw3bGgITLn68dajy0=; b=ST/Qtqrn1ChCkBZ60zkARnf+cXF65/lGkUPON4RNDXHBhLsELdEbwB67mVzuto9X6u R4LykjXDSXKaygGVqZDf7yX1fdBZb3KAiSU2QguQfPn5xwG3SOkklkMxUGVCnLMcadGV dw8Yfk+j9L67OZC+zgPMOGPKfkABvyMjHok0Un/aN8HPHu6QH2m1WLg5EXKjSKnT81Ap sbNMdjXYGy95CtcitYm3oIFe+e2LyPFot04K+8CTm7HMA96VyETAU4DR92t3XtKNVT8R lMm02g7nUeTqqxO/IxzjSRr8ovJEMCr/L7obN9koFoi4LWf25h+Ru7WH0KymfjK5sicY 6NCA== X-Gm-Message-State: AElRT7Hife5fMo2YcsMFHRj+CM4Kt8iadA07vq0i6C/xaLfSELTT4UE1 gUoSo/ip0YWDmAQyi1niIm2YL8LVImZb4F8JPXD7eA== X-Google-Smtp-Source: AG47ELvn8k1pOphLCTn9UCGpVEGKFh8CIcLp1G9z4OQaBDkAR7NePq+gB6psNsHBPRHYA9+TiOdvWShWVPlBstBIhS4= X-Received: by 10.107.58.134 with SMTP id h128mr41223496ioa.299.1522106465374; Mon, 26 Mar 2018 16:21:05 -0700 (PDT) MIME-Version: 1.0 Sender: wlosh@bsdimp.com Received: by 10.79.203.196 with HTTP; Mon, 26 Mar 2018 16:21:04 -0700 (PDT) X-Originating-IP: [2607:fb90:485c:937b:7dbf:9fc4:8aeb:59f] Received: by 10.79.203.196 with HTTP; Mon, 26 Mar 2018 16:21:04 -0700 (PDT) In-Reply-To: <2082659.Evq4QelJlA@ralph.baldwin.cx> References: <201803241540.w2OFeOcX037035@pdx.rh.CN85.dnsmgr.net> <1621825.K6AdhHRnSW@ralph.baldwin.cx> <1522088785.49673.25.camel@freebsd.org> <2082659.Evq4QelJlA@ralph.baldwin.cx> From: Warner Losh Date: Mon, 26 Mar 2018 16:21:04 -0700 X-Google-Sender-Auth: 8cl03YbkaSMFxvol8bcb7_jK_ko Message-ID: Subject: Re: svn commit: r331461 - in user/markj/netdump/sys: kern netinet/netdump sys vm To: John Baldwin Cc: Ian Lepore , "Rodney W. Grimes" , Mark Johnston , Andriy Gapon , src-committers@freebsd.org, svn-src-user@freebsd.org Content-Type: text/plain; charset="UTF-8" X-Content-Filtered-By: Mailman/MimeDel 2.1.25 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 23:21:07 -0000 On Mar 26, 2018 2:54 PM, "John Baldwin" wrote: On Monday, March 26, 2018 12:26:25 PM Ian Lepore wrote: > 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. That isn't fixed by having detailed commits in a WIP branch for compile fixes. To me that particular issue should be addressed by doing a better job importing things into HEAD including pulling out pieces of long-running branches and merging them as multiple commits each with useful commit messages. We should not rely on being able to find the history behind commits to HEAD because that history (if it exists) lives in random places. I have been doing longterm work with git rebase to curate the work for pushing to FreeBSD as the sub parts become available. For Lua thete were a hundred or so cleanup commits that preceded tthe main push it in commits (4 or 5 I think). Had I done a traditional svn branch, it would have been a huge dump all at once. By doing the 100 or so cleanup over several months, l found the mistakes in the small, bisectable bites earlier and easier than if I had pushed the whole mess in. Plus l developed a bunch of regression tests to make sure I caught the common rehressions the easrly cleanups caused. The actual development history was destoyed as i often did a messy cleanup on my lua branch, then transported diff hunks to another branch to polish up, push upstream then rebase the lua branch to erase the messy version. Early on, I published the git branch on github, but stopped becausenobody was testing or helping. That help didn't come until I pushed the 95% done loader sources with what I thought was 75% done lua menu things. Turns out Kyle Evans and a few others rewrote it in the month after my push, which makes me kijnda wish I'd pushed sooner... but it has worked out well... Warner