On Tue, Dec 15, 2015 at 10:55 AM, Joe Stringer <j...@ovn.org> wrote:

> On 14 December 2015 at 15:08, Andy Zhou <az...@ovn.org> wrote:
> > Fixes the following sparse warning messages:
> >
> >     lib/ovsdb-idl.c:146:12: error: symbol 'table_updates_names' was not
> >     declared. Should it be static?
> >     lib/ovsdb-idl.c:147:12: error: symbol 'table_update_names' was not
> >     declared. Should it be static?
> >     lib/ovsdb-idl.c:148:12: error: symbol 'row_update_names' was not
> >     declared. Should it be static?
> >
> > Reported-by: Joe Perches <j...@perches.com>
> > See: https://travis-ci.org/openvswitch/ovs/jobs/96737772#L870
> > Signed-off-by: Andy Zhou <az...@ovn.org>
>
> Did Joe Perches also report the error? Nice to know I'm not the only
> one checking Travis :-)
>
Sorry, cut-and-paste error.

>
> Just a note with the travis builds, you've already pointed out the
> failure in the commit message, so linking the failure doesn't provide
> much additional information. IMO It would provide more information if
> it is a build with this patch, so you can show that it fixes the
> build.
>
> O.K. I will drop the link.

> Acked-by: Joe Stringer <j...@ovn.org>
>
Thanks.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to