[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] block/curl: Implement the libcurl timer callbac
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH] block/curl: Implement the libcurl timer callback interface |
Date: |
Thu, 16 Jan 2014 09:55:18 +0000 |
On 16 January 2014 08:38, Paolo Bonzini <address@hidden> wrote:
> Il 15/01/2014 23:15, Peter Maydell ha scritto:
>>
>>> > + curl_multi_socket_action(s->multi, CURL_SOCKET_TIMEOUT, 0, &running);
>> The libcurl docs say "This function was added in libcurl 7.15.4, and
>> is deemed stable since 7.16.0. " So if we want to keep supporting
>> pre-7.16 libcurl then we need to retain the multi_socket_all codepath.
>>
>> On the other hand 7.16 was released in October 2006. What's
>> the oldest version we actually care about?
>
> I say 7.16 :)
What dos RHEL5 ship? That's usually our benchmark for
"oldest thing we need to support". Ubuntu 10.04 LTS (lucid)
and Debian oldstable (squeeze) both ship something more
recent than 7.16, so we're OK there.
We should probably update the configure test to check for
curl_multi_socket_action() rather than curl_multi_setopt().
thanks
-- PMM