svn commit: r347477 - head/sys/kern

Ian Lepore ian at freebsd.org
Sat May 11 14:45:23 UTC 2019


On Fri, 2019-05-10 at 23:57 -0500, Doug Moore wrote:
> With mentor approval, I commit r347469.  I start getting email about
> jenkins failure to build for several architectures on account of the
> _Generic() construct I introduced in that change.
> 
> I whip up a patch to undo that part of r347469, and ask for mentor
> approval.  Meanwhile, mentor authorizes me in email to revert
> r347469.
> 
> I try apply applying the fix-patch, and get email that it was
> rejected
> for lack of reviewer.  In retrospect, it seems to have been committed
> anyway as r347472.
> 
> Thinking that things are still broken, I do what my mentor pre-
> approved
> earlier and revert back to before r347469.  A patch to redo r347469,
> without _Generic(), awaits mentor approval.
> 
> I realize that breaking the build and then committing without mentor
> approval in my first week as committer isn't a good beginning.  
> Sorry
> about that.
> 
> At least I have no social media presence, so there's that.
> 
> Doug Moore
> 
> 

The important point is that the commit message should have said why. 
It doesn't have to be a whole novel, just something like "because
_Generic() isn't supported on all platforms" would've been good.

In general, for every commit, I ask myself "If someone 3 years from now
is debugging a problem and they're looking at the changes that have
happened to this code over time, will my commit message be helpful to
that process?"  Very often in that situation, what the person needs to
know isn't "what changed" so much as "why did this change happen?"

-- Ian


> On 5/10/19 11:47 PM, Cy Schubert wrote:
> > In message <201905110213.x4B2Dq9U088622 at repo.freebsd.org>, Doug
> > Moore 
> > writes:
> > > Author: dougm
> > > Date: Sat May 11 02:13:52 2019
> > > New Revision: 347477
> > > URL: https://svnweb.freebsd.org/changeset/base/347477
> > > 
> > > Log:
> > >   Revert r347469.
> > 
> > Why?
> > 
> > >   
> > >   Approved by: kib (mentor)
> > > 
> > > Modified:
> > >   head/sys/kern/subr_blist.c
> > > 
> 
> 
> 



More information about the svn-src-all mailing list