From owner-svn-src-head@freebsd.org Mon Dec 10 06:19:18 2018 Return-Path: Delivered-To: svn-src-head@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 675D7131317E; Mon, 10 Dec 2018 06:19:18 +0000 (UTC) (envelope-from freebsd@pdx.rh.CN85.dnsmgr.net) Received: from pdx.rh.CN85.dnsmgr.net (br1.CN84in.dnsmgr.net [69.59.192.140]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 66A838FEA7; Mon, 10 Dec 2018 06:19:17 +0000 (UTC) (envelope-from freebsd@pdx.rh.CN85.dnsmgr.net) Received: from pdx.rh.CN85.dnsmgr.net (localhost [127.0.0.1]) by pdx.rh.CN85.dnsmgr.net (8.13.3/8.13.3) with ESMTP id wBA6JB7m064610; Sun, 9 Dec 2018 22:19:11 -0800 (PST) (envelope-from freebsd@pdx.rh.CN85.dnsmgr.net) Received: (from freebsd@localhost) by pdx.rh.CN85.dnsmgr.net (8.13.3/8.13.3/Submit) id wBA6JB0c064609; Sun, 9 Dec 2018 22:19:11 -0800 (PST) (envelope-from freebsd) From: "Rodney W. Grimes" Message-Id: <201812100619.wBA6JB0c064609@pdx.rh.CN85.dnsmgr.net> Subject: Re: svn commit: r341759 - in head: contrib/wpa contrib/wpa/hostapd contrib/wpa/hs20/client contrib/wpa/src/ap contrib/wpa/src/common contrib/wpa/src/crypto contrib/wpa/src/drivers contrib/wpa/src/eap_c... In-Reply-To: To: Warner Losh Date: Sun, 9 Dec 2018 22:19:11 -0800 (PST) CC: Oliver Pinter , "Rodney W. Grimes" , svn-src-head@freebsd.org, Cy Schubert , svn-src-all@freebsd.org, src-committers , Cy Schubert Reply-To: rgrimes@freebsd.org X-Mailer: ELM [version 2.4ME+ PL121h (25)] MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset=US-ASCII X-Rspamd-Queue-Id: 66A838FEA7 X-Spamd-Result: default: False [2.15 / 15.00]; ARC_NA(0.00)[]; HAS_REPLYTO(0.00)[rgrimes@freebsd.org]; FROM_HAS_DN(0.00)[]; TO_DN_SOME(0.00)[]; NEURAL_SPAM_SHORT(0.38)[0.381,0]; NEURAL_HAM_LONG(-0.00)[-0.004,0]; MIME_GOOD(-0.10)[text/plain]; DMARC_NA(0.00)[dnsmgr.net]; REPLYTO_DOM_NEQ_FROM_DOM(0.00)[]; AUTH_NA(1.00)[]; NEURAL_SPAM_MEDIUM(0.91)[0.913,0]; RCVD_COUNT_THREE(0.00)[3]; TO_MATCH_ENVRCPT_SOME(0.00)[]; MX_GOOD(-0.01)[cached: pdx.rh.CN85.dnsmgr.net]; RCPT_COUNT_SEVEN(0.00)[8]; R_SPF_NA(0.00)[]; FROM_EQ_ENVFROM(0.00)[]; R_DKIM_NA(0.00)[]; RCVD_TLS_LAST(0.00)[]; ASN(0.00)[asn:13868, ipnet:69.59.192.0/19, country:US]; MID_RHS_MATCH_FROM(0.00)[]; IP_SCORE(-0.03)[asn: 13868(-0.05), country: US(-0.09)] X-Rspamd-Server: mx1.freebsd.org X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.29 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: Mon, 10 Dec 2018 06:19:18 -0000 > On Sun, Dec 9, 2018 at 2:03 PM Oliver Pinter > wrote: > > > > > > > On Sunday, December 9, 2018, Warner Losh wrote: > > > >> On Sun, Dec 9, 2018 at 1:09 PM Rodney W. Grimes < > >> freebsd@pdx.rh.cn85.dnsmgr.net> wrote: > >> > >> > > In message <201812090645.wB96jnso066329@repo.freebsd.org>, Cy > >> Schubert > >> > > writes: > >> > > > Author: cy > >> > > > Date: Sun Dec 9 06:45:49 2018 > >> > > > New Revision: 341759 > >> > > > URL: https://svnweb.freebsd.org/changeset/base/341759 > >> > > > > >> > > > Log: > >> > > > MFV r341618: > >> > > > > >> > > > Update wpa 2.6 --> 2.7. > >> > > > >> > > Relnotes: yes > >> > > >> > As an FYI, or maybe a new procedure, doing a reply to > >> > a commit message adding relnotes: yes does very little > >> > to insure that this commit gets refered to in release > >> > notes. > >> > > >> > What about we add RELNOTES.missed to the tree > >> > next to UPDATING, and when someone forgets to tag > >> > the Relnotes:yes into a commit you just follow up > >> > with a commit to that file, stating the svn revision > >> > which was missing the note and then we have a nice > >> > documented and clean way to extract the missing > >> > release note items, rather than trying to cull it > >> > from a thread in a mail list archive. > >> > > >> > The file would get truncated to 0 at appropriate > >> > times on various branches. > >> > > >> > >> How about just RELNOTES. You put the revision that is relevant and a quick > >> blurb. That way we don't have to look in two places. All release notes go > >> in here, no exceptions. You can retroactively tag them, or you can commit > >> this as part of commit. My one concern is that if we remove the Relnotes: yes line from the commits then we may end up totally ignoring the need to put entries in RELNOTES, which unlike UPDATING do not have consequences if ignored. > > > > > >> > > I don't really know SVN, but there wouldn't be a chicken egg probem during > > commit time? I mean you would really know the SVN id. So you could only add > > a specific revision in a different commit to RELEASE file. > > > > Generally, you can guess really well, and fix it in the case of a lost race > easily. No reason to guess, if you add the RELNOTES change with the commit then it is the revision that added the RELNOTES entry, so a log view of RELNOTES would show you the revision. If you do it after words or edit an existing entry in the RELNOTES file that is also fairly clear as that commit has no other files touched. > > You'd add the release notes text in full to the file, with a pointer to the > revision(s) for the feature. If you modify RELNOTES with the commit adding the feature we could easily use a pointer of "this commit", the svn version number of that added entry is self referencing to the actual change, which I actually rather like the idea of. > > Warner > > > > >> Have a blurb at the top that tells people what > >> order to add them in, and you'd be set. We'd then retire "relnotes: yes" > >> in > >> the commit message. This would also allow 'helpers' to format the RELNOTES > >> file as we go rather than having to play 2 years of catch-up at major > >> branch times. Yes. You could actually "delete" an entry from RELNOTES once a proper entry in the actual release notes had been created, such that RELNOTES is really a list of pending items. > >> > >> Having it in the commit message just doesn't work, and this is one of many > >> reasons: Cy forgot. Other times I'll do something and it's only a month > >> later I realize it needs to be in the release notes after some issue has > >> come up.... Other times I put relnotes: yes in only to realize that's my > >> vanity talking, and nobody else cares. I agree, what we have now works poorly. > >> Warner -- Rod Grimes rgrimes@freebsd.org