svn commit: r293720 - head/sys/dev/hyperv/netvsc

[Alexey Dokuchaev]

On Tue, Jan 12, 2016 at 07:30:05AM -0700, Ian Lepore wrote:
> On Tue, 2016-01-12 at 08:55 +0100, Joel Dahl wrote:
> > On Tue, Jan 12, 2016 at 01:41:34AM +0000, Sepherosa Ziehau wrote:
> > > Author: sephe
> > > Date: Tue Jan 12 01:41:34 2016
> > > New Revision: 293720
> > > URL: https://svnweb.freebsd.org/changeset/base/293720
> > > 
> > > [...]
> > >   Modified by:		me (minor)
> > 
> > % finger me at freebsd.org
> > [freebsd.org]
> > Trying 8.8.178.110...
> > Login: me             			Name: Michael Elbel
> 
> Really?  You're complaining about the committer using the personal
> pronoun that refers to one's self in a context of referring to himself?

I understand you; with certain entries like `me' or `myself' it does look
silly indeed, however...

> Do we really, as a project, have nothing better to do than this endless
> nitpicking at the metadata in commit messages?

... many of us neglect to give enough thinking and thus often give bogus
attribution in the logs; while complaining over personal pronouns is off
the line, the problem still stands, and I'm glad that people are paying
attention to these things.  Please don't be hard on Joel.

Generally, "Submitted by: foo (modified)" should be enough metadata to
reflect the fact that submitted changes were not applied verbatim; which
in turn is appropriate amendment for more of less substantial changes,
trivial modifications do not warrant it.

./danfe