[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