[libvirt] [PATCH 6/9] networkxml2conftest: s/lo/lo0/ on non-Linux
Michal Privoznik
mprivozn at redhat.com
Mon Jan 2 10:02:02 UTC 2017
On 01.01.2017 17:14, Roman Bogorodskiy wrote:
> Michal Privoznik wrote:
>
>> After 478ddedc12 a bug is fixed where we wrongly presumed loopack
>> device name on non-Linux systems. It's lo0. However, the fix is
>> not reflected in the tests which are failing now.
>>
>> Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
>> ---
>> tests/networkxml2conftest.c | 15 +++++++++++++--
>> 1 file changed, 13 insertions(+), 2 deletions(-)
>>
>> diff --git a/tests/networkxml2conftest.c b/tests/networkxml2conftest.c
>> index a80d3b2d4..404e2c722 100644
>> --- a/tests/networkxml2conftest.c
>> +++ b/tests/networkxml2conftest.c
>> @@ -41,10 +41,21 @@ testCompareXMLToConfFiles(const char *inxml, const char *outconf, dnsmasqCapsPtr
>> if (dctx == NULL)
>> goto fail;
>>
>> - if (networkDnsmasqConfContents(obj, pidfile, &actual,
>> - dctx, caps) < 0)
>> + if (networkDnsmasqConfContents(obj, pidfile, &actual, dctx, caps) < 0)
>> goto fail;
>>
>> +#ifndef __linux__
>> + char * tmp;
>> +
>> + if (!(tmp = virStringReplace(actual,
>> + "except-interface=lo0\n",
>> + "except-interface=lo\n")))
>> + goto fail;
>> + VIR_FREE(actual);
>> + actual = tmp;
>> + tmp = NULL;
>> +#endif
>> +
>> if (virTestCompareToFile(actual, outconf) < 0)
>> goto fail;
>
> Michal, what do you think about this version of this fix:
>
> https://www.redhat.com/archives/libvir-list/2016-December/msg01190.html
>
That approach looks like heavy hammer to me. And as Andrea pointed out,
my approach works even with VIR_TEST_REGENERATE_OUTPUT.
Michal
More information about the libvir-list
mailing list