[libvirt] [PATCH 3/3] networkxml2conftest: Fix build on BSD

Michal Privoznik mprivozn at redhat.com
Mon Jan 2 12:10:00 UTC 2017


On 02.01.2017 12:04, Martin Kletzander wrote:
> On Mon, Jan 02, 2017 at 09:57:04AM +0100, Andrea Bolognani wrote:
>> On Sun, 2017-01-01 at 12:35 +0400, Roman Bogorodskiy wrote:
>>> > No rush, I see Michal has yet another proposal for this that we
>>> haven't
>>> > considered and even though there are somedrawbacks to that as well, it
>>> > looks nicer than this.
>>> > 
>>> > After all the ideas I'm starting to like the "gross" one the best.  Oh
>>> > my =)
>>>  
>>> My vote still goes to this solution, because having a placeholder seems
>>> more explicit and easier to follow than doing s/lo/lo0/ directly.
>>
>> I vote for Michal's approach as it doesn't require us to
>> disable VIR_TEST_REGENERATE_OUTPUT.
>>
> 
> Well, it does.  Kinda.  You *must not* regenerate output on FreeBSD with
> his patch, so it should be explicitly disabled.

Really? I think it works well even if you do regenerate output there. I
mean, my patch fixes the output of the actual configuration, so that it
will always contain 'lo' instead of 'lo0'. And test output regeneration
is done after that. With 'lo'.

Michal




More information about the libvir-list mailing list