Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 12 Jan 2016 15:28:00 +0000
From:      Alexey Dokuchaev <danfe@FreeBSD.org>
To:        Ian Lepore <ian@freebsd.org>
Cc:        Joel Dahl <joel@vnode.se>, Sepherosa Ziehau <sephe@FreeBSD.org>, svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r293720 - head/sys/dev/hyperv/netvsc
Message-ID:  <20160112152800.GA94850@FreeBSD.org>
In-Reply-To: <1452609005.46848.11.camel@freebsd.org>
References:  <201601120141.u0C1fYbC080237@repo.freebsd.org> <20160112075525.GA38087@ymer.vnode.se> <1452609005.46848.11.camel@freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
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@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



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20160112152800.GA94850>