Re: [libvirt] Magic error introduced by commit f09accc

From: Jiri Denemark <jdenemar redhat com>

To: Eric Blake <eblake redhat com>

Cc: libvir-list redhat com

Subject: Re: [libvirt] Magic error introduced by commit f09accc

Date: Mon, 9 May 2011 11:05:08 +0200

On Fri, May 06, 2011 at 16:33:13 -0600, Eric Blake wrote:
> On 05/06/2011 01:18 PM, Jiri Denemark wrote:
> We discussed more on IRC. The problem is due to a bug in out-of-the-box
> libnl headers, and has been patched in most distros to just delete the
> 'inline' as then you don't have to worry about whether you had old or
> new gcc semantics for 'extern inline'. That is, libvirt could probably
> work around the issue by doing:
>
> #define inline
> #include <netlink/msg.h>
> #undef inline
>
> in src/util/macvtap.c.
>
> However, I don't know if we should do that.
I think we shouldn't do this.
> In one regards, the only people that ever run into this are those that are
> using incompatible self-built gcc and libnl (Jirka was testing on a gentoo
> system), and we're doing the user a favor to have them patch their libnl
> headers rather than making libvirt work around it. Most distros (including
> RHEL 5, which stands as the current litmus test of how much workarounds we
> have to support in libvirt) have already patched the header
And I filed a BZ for gentoo as well:
http://bugs.gentoo.org/show_bug.cgi?id=366561
Jirka