Date: Tue, 12 Mar 2019 08:43:00 -0700 From: John Baldwin <jhb@FreeBSD.org> To: Warner Losh <imp@bsdimp.com>, "Rodney W. Grimes" <rgrimes@freebsd.org> Cc: Warner Losh <imp@freebsd.org>, src-committers <src-committers@freebsd.org>, svn-src-all <svn-src-all@freebsd.org>, svn-src-head <svn-src-head@freebsd.org> Subject: Re: svn commit: r345050 - head/bin/date Message-ID: <c308a5ee-40f8-16e1-b24a-9037c3237898@FreeBSD.org> In-Reply-To: <CANCZdfoHFNApkSizaQFhQNgZw7H1-XAVL%2BDD4M9rsvguFJL2cA@mail.gmail.com> References: <201903120515.x2C5Fbtw013485@gndrsh.dnsmgr.net> <201903120525.x2C5PH7X013552@gndrsh.dnsmgr.net> <CANCZdfoHFNApkSizaQFhQNgZw7H1-XAVL%2BDD4M9rsvguFJL2cA@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On 3/11/19 10:43 PM, Warner Losh wrote: > On Mon, Mar 11, 2019 at 11:25 PM Rodney W. Grimes <freebsd@gndrsh.dnsmgr.net> > wrote: > >>>> Author: imp >>>> Date: Tue Mar 12 04:49:59 2019 >>>> New Revision: 345050 >>>> URL: https://svnweb.freebsd.org/changeset/base/345050 >>>> >>>> Log: >>>> Remove now useless -d and -t flags. >>>> >>>> These were used to set dst flag and minutes west of UTC >>>> respectively. These are obsolete and have been removed form the >>>> kernel. These existed primarily to faithfully emulate early >>>> Unix ABIs that have been removed from FreeBSD. >>>> >>>> Reviewed by: jbh@, brooks@ >> Nits: jhb@ and I see he did comment in the review, but he did not >> accept it as a reviewew at the top. >> > > This is why I think just the reference to the differential revision is > perfectly fine. Why duplicate data? Others complained I hadn't included it. > He commented, we discussed it on irc (though most of it was about how to > use arc better), etc. I thought it warranted it. So please don't nitpick. > This level is really annoying and frustrating. Does this really help us > produce a better product? To be fair, we have been pretty consistent that 'Reviewed by' in the commit means 'Accepted' in phab. I probably would have ended up accepting this anyway and just didn't click the box, so it's ok. However, we have had folks in the past who were tagged in the past and gave feedback, but didn't approve of the change, but were listed as 'Reviewed by' hence the current practice. -- John Baldwin
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?c308a5ee-40f8-16e1-b24a-9037c3237898>