svn commit: r317409 - head/contrib/tcpdump

Benjamin Kaduk bjkfbsd at gmail.com
Tue Apr 25 18:33:40 UTC 2017


On Tue, Apr 25, 2017 at 1:30 PM, Rodney W. Grimes <
freebsd at pdx.rh.cn85.dnsmgr.net> wrote:

> [ Charset UTF-8 unsupported, converting... ]
> > Author: glebius
> > Date: Tue Apr 25 15:56:46 2017
> > New Revision: 317409
> > URL: https://svnweb.freebsd.org/changeset/base/317409
> >
> > Log:
> >   Cherry-pick 5d3c5151c2b885aab36627bafb8539238da27b2d, it fixes use
> after free
>
> Lets not use git hashes as references in commit messages,
> it has very little value for someone reading a log of changes
> looking for something.  There is no promise that it can be
> found later (github can go Poof).
>

On the contrary, a git SHA1 seems like an eminently stable and unique
search parameter!
I agree that a commit log should inline some summary of the change as well
as provide a
link to the external source, but I am not worried that a future reader will
be unable to find
the referenced commit.

-Ben


More information about the svn-src-head mailing list