From owner-svn-src-stable-11@freebsd.org Thu Mar 15 17:23:35 2018 Return-Path: Delivered-To: svn-src-stable-11@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 76A292E for ; Thu, 15 Mar 2018 17:23:35 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: from mail-it0-x230.google.com (mail-it0-x230.google.com [IPv6:2607:f8b0:4001:c0b::230]) (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 B4ECF6CE24 for ; Thu, 15 Mar 2018 17:23:34 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: by mail-it0-x230.google.com with SMTP id u5-v6so10133904itc.1 for ; Thu, 15 Mar 2018 10:23:34 -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=CwV+qZjHw27uCHSzvj9+6K6a6eH6pjkAtY9bY+1MrpA=; b=P04QxGKtrJwuHS43j/mFpK+jBXZ4OTuMe7VBPqg95IBylTHzoKLMgOmZKCmcO2eNvh 7ACY2EzC7rEP/2lSoA1QErtD/VG+B9flnMxY8ZKBGabj1IpFK7P5hfTD/Bw15q2ySafa AQZ/viHH5p3ovtYt0rc3SAtyeseVwr0Ruh/laacc3mlAlzrZkkyMzAy3Yn2Tkcv9f7IF 4c+W9atJNniyA4xQieUIWEjfZ09l3jUXGeHcvjKghkIZbYma6utrWLPeWzxD292Vlrem UTkuQibd43ij4g5akFu84H9PvOoNiyIiZQQ70/QR3/G8UNIncj/wgzmWc6Bc5fWUub80 TUow== 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=CwV+qZjHw27uCHSzvj9+6K6a6eH6pjkAtY9bY+1MrpA=; b=uVJOThXeqU6MZmJPkrg9gaIbqAsgSMJKgR80edXItRY8/+++4LzM1QotW8IuVYPPb2 GY0W4gfA3gjfkzK2dVf+zgUGSPXLTF7UvDq9Kx4uVonL5mK0xd4QBGGexCHZFJhqy9n0 WXQGm0Jw4O/dZXFnfSJKPNBuTXDAr9oCXzw2caVmhOxKyfIkd+PKfrUlSfqKMx4SP6fR bBKvevOaU1e4N3yb4+n0KNjlRbSEHYp/QFvIHqO6hn3Ii08T0RQC485A7ZnBIHAbdC5G cLcFUs9fueRVm7qWR/oH1R6+UOX/XOQJJNcNNAQrMz58rmoyWqwNtabmSCtX9NPq2N79 b0hw== X-Gm-Message-State: AElRT7H5Qqj5PsyD06IJg9CoAvV+mbuKjA/7LBc30jgWo1Lc1Lssmvqy MAdplrSaSDnSuIUs60hJErWNxMgmO5TB1ARy9+hzXg== X-Google-Smtp-Source: AG47ELsa74maVx/lFU5vLUa544SqfPNvPPyULhJfeEDzhWZfYs4dK/zzqIPp8ssCs4WvstzlUciQvGvwtCqDm955gQk= X-Received: by 10.36.66.134 with SMTP id i128mr2171393itb.73.1521134612742; Thu, 15 Mar 2018 10:23:32 -0700 (PDT) MIME-Version: 1.0 Sender: wlosh@bsdimp.com Received: by 10.79.203.196 with HTTP; Thu, 15 Mar 2018 10:23:31 -0700 (PDT) X-Originating-IP: [2603:300b:6:5100:1052:acc7:f9de:2b6d] In-Reply-To: References: <201803151614.w2FGEttu093805@pdx.rh.CN85.dnsmgr.net> <1521130827.99081.43.camel@freebsd.org> From: Warner Losh Date: Thu, 15 Mar 2018 11:23:31 -0600 X-Google-Sender-Auth: btlF0CUDO6ZcKUhFhxgPRKcqHpU Message-ID: Subject: Re: svn commit: r330972 - stable/11/share/misc To: Oliver Pinter Cc: Ian Lepore , src-committers , Eitan Adler , Justin Hibbits , "svn-src-stable@freebsd.org" , "svn-src-all@freebsd.org" , "svn-src-stable-11@freebsd.org" , Andriy Gapon , "Rodney W. Grimes" X-Mailman-Approved-At: Thu, 15 Mar 2018 19:06:42 +0000 Content-Type: text/plain; charset="UTF-8" X-Content-Filtered-By: Mailman/MimeDel 2.1.25 X-BeenThere: svn-src-stable-11@freebsd.org X-Mailman-Version: 2.1.25 Precedence: list List-Id: SVN commit messages for only the 11-stable src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 15 Mar 2018 17:23:35 -0000 On Thu, Mar 15, 2018 at 11:07 AM, Oliver Pinter < oliver.pinter@hardenedbsd.org> wrote: > > > On Thursday, March 15, 2018, Warner Losh wrote: > >> On Thu, Mar 15, 2018 at 10:31 AM, Warner Losh wrote: >> >> > >> > >> > On Thu, Mar 15, 2018 at 10:20 AM, Ian Lepore wrote: >> > >> >> On Thu, 2018-03-15 at 09:14 -0700, Rodney W. Grimes wrote: >> >> > > >> >> > > On Thu, 2018-03-15 at 10:52 -0500, Justin Hibbits wrote: >> >> > > > >> >> > > > On Thu, Mar 15, 2018 at 10:46 AM, Ian Lepore >> >> > > > wrote: >> >> > > > > >> >> > > > > >> >> > > > > I agree completely with all of this.??It bothers me how many >> >> > > > > committers >> >> > > > > have the attitude that handling MFCs is not part of being a >> >> > > > > committer. >> >> > > > Never attribute to arrogance that which can adequately be >> >> > > > explained >> >> > > > by >> >> > > > sheer laziness ;) >> >> > > > >> >> > > > - Justin (guilty of marking changes as MFC after, and ignoring >> >> > > > them >> >> > > > for far too long) >> >> > > > >> >> > > Laziness and procrastination I understand -- I own a lovely glass >> >> > > house >> >> > > in that neighborhood. ?I tend to put off MFCs for way too long then >> >> > > every few months have to spend a whole weekend catching up. >> >> > MFC: 1 week (by pool|self) #defaults to self if missing >> >> > >> >> > There is already a very nice tracking tool for outstanding MFC's, >> >> > if we added a bit of smarts in its parser, and created a pool of >> >> > MFC commiters (Eitan seems to have started one :-)) those who >> >> > do not want to do there own MFC work could pass the hat. >> >> >> >> If you're talking about the MFC after: field in commits, I don't use >> >> it. I have about zero tolerance for being nagged by anybody about >> >> anything, and that goes double for robots nagging me with spam mail. >> >> >> >> The MFC tool that works well for me is gonzo's MFCTracker site [*] that >> >> doesn't require extra markup in the commit messages. >> >> >> > >> > I also have a MFC tool for git, but it's n >> > >> >> [[ stupid track pad and too easy button pushes... ]] >> >> but it's not ready for prime time. It's useful if you have a list of >> things >> you want to MFC for playing them onto the stable branch so you can test >> before committing to svn stable. It shows the big issues with moving to >> git >> as the source of truth, though. We have way too much traffic in the repo >> to >> have git cherry to produce any kind of reasonable output (too many >> changes, >> can't restrict to a subset of the tree, no way to check prior commits to >> files affected, etc), and the git cherry-pick command relies a bit too >> much >> on the merge magic, so it doesn't record merges (there is no merge-info in >> git). >> >> However, I could dust off the tool and fix up the rough edges if there's >> any interest at all. Kyle Evans used it to MFC my crazy src/stand stuff... >> >> > I use this script to merge / cherry-pick changes from master: > https://github.com/opntr/opBSD-ng-tools/blob/master/git/opBSD_mfc.sh > Nice. My own script is at https://people.freebsd.org/~imp/git-mfc though it could learn a thing or two from yours in some ways (mine is driven not by git has, but by svn #). Not, this is a quick-rehash of a script we use internally to cherry pick stuff to the Netflix oca firmware, so there's some stale comments in it that point to internal docs. I'll have to rework it with the git show trick. Warner