<tt><font size=2>Matthias Bolte <matthias.bolte@googlemail.com>
wrote on 06/23/2011 03:17:32 AM:<br>
<br>
> 2011/6/23 Laine Stump <laine@laine.org>:<br>
> > On 06/22/2011 06:42 PM, Matthias Bolte wrote:<br>
> >><br>
> >> The refactoring in 6a5978833a5 and df3d8c362d3 was incomplete<br>
> >> as it accidentally moved macvtap related code out of a<br>
> >> #if WITH_MACVTAP in a #if __linux__ block. To fix this move<br>
> >> ifaceMacvtapLinkAdd and ifaceMacvtapLinkDump back under<br>
> >> #if WITH_MACVTAP.<br>
> >><br>
> >> Also nlComm was moved from #if WITH_MACVTAP to #if __linux__<br>
> >> but configure.ac was not updated to match this, as libnl
is<br>
> >> now required on Linux always because of this.<br>
> ><br>
> > So with your changes it's still required for all Linux builds?
That's<br>
> > potentially not good at all. RHEL5 uses libnl 1.0, which is not
ABI<br>
> > compatible with libnl 1.1 (which everything in libvirt is written
to). (Up<br>
> > to now we've been saved the headache because macvtap isn't enabled/supported<br>
> > on RHEL5.)<br>
> <br>
> Well, the whole problem was introduced by Stefans cleanup commits.
Is<br>
> there some higher level reason for this refactoring? There probably<br>
> is, but If not then the simplest fix is to revert the offending<br>
> commits.</font></tt>
<br>
<br><tt><font size=2>I apologize for this. I am fixing this now.</font></tt>
<br>
<br><tt><font size=2>   Stefan</font></tt>
<br>