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

Ben Pfaff blp at nicira.com
Mon Jun 14 23:17:04 UTC 2010


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?




More information about the dev mailing list