[ovs-dev] [PATCH 2/2] configure: add configure option to disable building brcompat

[Chris Wright]

* Ben Pfaff (blp at nicira.com) wrote:
> On Fri, Mar 09, 2012 at 06:17:07PM -0800, Chris Wright wrote:
> > This adds ability to do:
> > 
> >   ./configure --disable-brcompat
> > 
> > to disable building userspace and kernel module associated with
> > providing linux bridge compatibility.
> > 
> > Signed-off-by: Chris Wright <chrisw at sous-sol.org>
> 
> I've had a little bit of trouble in the past with Automake not
> distributing some kinds of files that are conditionally built.  So,
> with this patch applied, does
>         ./configure --disable-brcompat
>         make distcheck
> pass, e.g. does "make dist" still distribute the source files and
> (especially) the manpages that are now conditional?

Yes, you're right, it does seem to effect the datapath.

--- enable.out	2012-03-12 11:15:46.083861997 -0700
+++ disable.out	2012-03-12 11:15:37.362738804 -0700
@@ -141,7 +141,6 @@
 openvswitch-1.6.90/datapath/linux/Makefile.main.in
 openvswitch-1.6.90/datapath/linux/Module-brcompat.mk
 openvswitch-1.6.90/datapath/linux/compat/
-openvswitch-1.6.90/datapath/linux/compat/genetlink-brcompat.c
 openvswitch-1.6.90/datapath/linux/compat/flex_array.c
 openvswitch-1.6.90/datapath/linux/compat/workqueue.c
 openvswitch-1.6.90/datapath/linux/compat/kmemdup.c
@@ -224,7 +223,6 @@
 openvswitch-1.6.90/datapath/datapath.c
 openvswitch-1.6.90/datapath/compat.h
 openvswitch-1.6.90/datapath/vport-internal_dev.h
-openvswitch-1.6.90/datapath/brcompat_main.c
 openvswitch-1.6.90/datapath/dp_sysfs.h
 openvswitch-1.6.90/datapath/vport-netdev.c
 openvswitch-1.6.90/datapath/vport.h