[ovs-dev] [PATCH 2/2] vlog: Check that all declared vlog modules are used, at "make dist" time.

Jesse Gross jesse at nicira.com
Tue Jun 15 00:10:05 UTC 2010


On Mon, Jun 14, 2010 at 4:17 PM, Ben Pfaff <blp at nicira.com> wrote:

> On Mon, Jun 14, 2010 at 01:24:09PM -0700, Jesse Gross wrote:
> > One thought that did come to mind is whether this hook is really best
> suited
> > as a dist hook, as it doesn't really have anything to do with
> distribution.
> >  It doesn't really matter though, since the presence of extra vlog
> modules
> > doesn't hurt anything and "make dist" should be run often enough for
> > somebody to notice.
>
> We *could* hang it off of the "all" target (using "all-local" instead of
> "dist-hook").  Then we'd get it checked on every "make".  Does that
> sound reasonable?
>

Sure, that sounds good.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openvswitch.org/pipermail/ovs-dev/attachments/20100614/b4175104/attachment-0003.html>


More information about the dev mailing list