[libvirt] [PATCH] Fix compilation of statstest.c during make check
Jim Fehlig
jfehlig at novell.com
Thu Jul 7 16:44:07 UTC 2011
Eric Blake wrote:
> On 07/07/2011 10:16 AM, Jim Fehlig wrote:
>
>> ---
>> tests/statstest.c | 2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/tests/statstest.c b/tests/statstest.c
>> index d18bb0c..c989992 100644
>> --- a/tests/statstest.c
>> +++ b/tests/statstest.c
>> @@ -17,7 +17,7 @@ static void testQuietError(void *userData ATTRIBUTE_UNUSED,
>>
>> static int testDevice(const char *path, int expect)
>> {
>> - int actual = xenLinuxDomainDeviceID(NULL, 1, path);
>> + int actual = xenLinuxDomainDeviceID(1, path);
>>
>
> ACK.
>
Pushed.
> This was a regression introduced in commit 60d769a1, back in April. So
> why did we not detect it until now? What is different about your test
> environment that triggered compilation of this code while others (myself
> included) have been happily unaware of its existence?
>
In configure.ac, I see that xen kernel and xend must both be running to
for ENABLE_XEN_TESTS to be true, which then enables building statstest.c
and reconnect.c. I guess we're just not using xen much these days...
Regards,
Jim
More information about the libvir-list
mailing list