svn commit: r344129 - head
imp at bsdimp.com
Fri Feb 15 03:09:07 UTC 2019
On Thu, Feb 14, 2019, 1:42 PM Rodney W. Grimes <
freebsd at pdx.rh.cn85.dnsmgr.net wrote:
> > On Thu, Feb 14, 2019 at 12:00:22PM -0700, Warner Losh wrote:
> > > On Thu, Feb 14, 2019 at 11:29 AM Rodney W. Grimes <
> > > > > Differential Review: https://reviews.freebsd.org/D19193
> > > >
> > > > You sited a differential, but not give any attribution
> > > > to the external source :-(
> > > >
> > >
> > > The differential review has that information.
> > External contributors should be recognized by having their names appear
> > in the commit logs.
> We even bother to put a special line in the commit template
> for this. Further it has been standard operating procedure
> for at least as long as I have been back that submitters
> are infact recognized in commit messages.
> You have, again, summarily dismissed valid feedback.
The problem is that was a trivial commit. And you offered not one, but two
complaints about adding an 'e' to the updating file. That's what pissed me
off. It's advice that might be correct, but was so far over the top, in
public, for such a trivial commit. That's why I got mad: it added no value
and seemed nit picky and pretty. So I lost it. I shouldn't have, but I did.
I'm sorry for my cool.
More information about the svn-src-all