I'm fine with that. Wasn't sure what's the stylistically correct way to implement a terrible hack. I'll resend it.
Ethan On Fri, Sep 2, 2011 at 11:56, Jesse Gross <[email protected]> wrote: > On Thu, Sep 1, 2011 at 7:53 PM, Ethan Jackson <[email protected]> wrote: >> OVS fails to build with xenddk-56100build3926 because it has an >> outdated genetlink header. >> --- >> lib/netlink-protocol.h | 11 +++++++++++ >> 1 files changed, 11 insertions(+), 0 deletions(-) >> >> diff --git a/lib/netlink-protocol.h b/lib/netlink-protocol.h >> index 1b5fa71..6a3b92d 100644 >> --- a/lib/netlink-protocol.h >> +++ b/lib/netlink-protocol.h >> @@ -36,6 +36,17 @@ >> #ifdef HAVE_NETLINK >> #include <linux/netlink.h> >> #include <linux/genetlink.h> >> + >> +/* Some Xenddks have an outdated genetlink header file which doesn't have >> some >> + * symbols it should. */ >> +#ifndef CTRL_ATTR_MCAST_GROUP_MAX >> +#define CTRL_ATTR_MCAST_GRP_NAME 1 >> +#define CTRL_ATTR_MCAST_GRP_ID 2 >> +#define CTRL_ATTR_MCAST_GROUPS 7 >> +#undef CTRL_ATTR_MAX >> +#define CTRL_ATTR_MAX 7 >> +#endif /* CTRL_ATTR_MCAST_GROUP_MAX */ > > I would probably just drop in the entire CTRL_ATTR_MCAST_* enum and > define but this should work as well. > _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
