Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 26 Apr 2017 14:51:14 -0700 (PDT)
From:      "Rodney W. Grimes" <freebsd@pdx.rh.CN85.dnsmgr.net>
To:        Gleb Smirnoff <glebius@FreeBSD.org>
Cc:        rgrimes@FreeBSD.org, src-committers@FreeBSD.org, svn-src-all@FreeBSD.org,  svn-src-head@FreeBSD.org
Subject:   Re: svn commit: r317409 - head/contrib/tcpdump
Message-ID:  <201704262151.v3QLpEMX068593@pdx.rh.CN85.dnsmgr.net>
In-Reply-To: <20170426212305.GF56922@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
>   Rodney,
> 
> On Tue, Apr 25, 2017 at 11:30:17AM -0700, Rodney W. Grimes wrote:
> R> > Author: glebius
> R> > Date: Tue Apr 25 15:56:46 2017
> R> > New Revision: 317409
> R> > URL: https://svnweb.freebsd.org/changeset/base/317409
> R> > 
> R> > Log:
> R> >   Cherry-pick 5d3c5151c2b885aab36627bafb8539238da27b2d, it fixes use after free
> R> 
> R> Lets not use git hashes as references in commit messages,
> R> it has very little value for someone reading a log of changes
> R> looking for something.  There is no promise that it can be
> R> found later (github can go Poof).
> 
> Since I did previous import of tcpdump, I find it very useful to
> see what exactly has been cherry-picked forward, for the person
> who will do next import, be it me or someone else.
> 
> I agree that I should have put the hash into "meta data portion
> of commit message", not in the description. I will do so next time.

Thank you!  Thats really all I wanted

-- 
Rod Grimes                                                 rgrimes@freebsd.org



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