On Mon, Aug 13, 2018 at 01:16:25PM -0400, Aaron Conole wrote: > Ben Pfaff <[email protected]> writes: > > > On Sun, Aug 12, 2018 at 07:43:14AM -0700, Ben Pfaff wrote: > >> On Sat, Aug 11, 2018 at 09:54:01AM -0400, Aaron Conole wrote: > >> > Ben Pfaff <[email protected]> writes: > >> > > >> > > This also makes a start at a testsuite for checkpatch. > >> > > > >> > > CC: Aaron Conole <[email protected]> > >> > > Signed-off-by: Ben Pfaff <[email protected]> > >> > > --- > >> > > >> > Acked-by: Aaron Conole <[email protected]> > >> > > >> > Thanks for this work, Ben! > >> > > >> > There's one more case that we could possibly consider (but it's rare > >> > enough that maybe it's not that big of a deal to let it stand for now), > >> > encompassed by commit 52e20a3d6c8b. > >> > > >> > Once this goes in I'll fix the bot up to add a signoff (so that the > >> > warning is squelched :). > >> > >> I guess another way would be to make checkpatch ignore some particular > >> committer, e.g. "Nobody <[email protected]>" and not complain about the > >> lack of sign-off. > > > > Or the robot could just use the author as committer as well. > > I'll use that option.
OK. I applied this to master. Thank you for all the reviews! _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
