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

Martin Kletzander mkletzan at redhat.com
Mon Jan 2 11:04:31 UTC 2017


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.

I can't make up my mind, currently I'm inclining to your solution with
multiple files, so I'll let you guys decide.

>-- 
>Andrea Bolognani / Red Hat / Virtualization
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20170102/a3a6ed62/attachment-0001.sig>


More information about the libvir-list mailing list