On Sun, May 8, 2016 at 8:59 AM, Ben Pfaff <b...@ovn.org> wrote:

> On Fri, May 06, 2016 at 07:44:57PM -0700, Darrell Ball wrote:
> > On Fri, May 6, 2016 at 5:57 PM, Justin Pettit <jpet...@ovn.org> wrote:
> >
> > >
> > > > On May 4, 2016, at 12:03 PM, Darrell Ball <dlu...@gmail.com> wrote:
> > > >
> > > > This patch series updates the vtep schema, vtep-ctl commands and vtep
> > >
> > > It's not really a series.  Even if it were, when looking at the commit
> > > history once its committed, it wouldn't show as a series.
> > >
> >
> > I actually did not think it mattered.
> > series == set, which could have a single member :-)
> > I'll remove the series part.
>
> I think Justin's point is that series (regardless of their length)
> aren't apparent once they've been committed to the Git repository, since
> there's nothing to link them together at that point, which makes it
> confusing to refer to them in commit messages.
>
> It does make total sense to refer to series in the commentary that
> accompanies a patch (following ---), but that doesn't end up in the
> commit message.
>

Thanks Ben
I still need to keep a closer eye on "---" it seems :-)
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to