On 21/09/11 13:47, Mike Dupont wrote:
> I've
been working on a replacement for the libcurl patch.
> -- Tim Starling
Thats Good news,
Last night I removed all traces of the curl_multi_select from the code, no
idea if it runs properly, but it produces a package. Also I commented out
all the non standard libevent code.
The package has a compiler that produces code, and you can at least
experiment with it.
We need more people testing, I am getting help from somone from the debian
team as well.
Will continue working on this as I find time.
curl_multi_select() may be needed by someone, it is a standard PHP
function. Removing it will work as long as no user code calls it.
The implementation in Zend is simple and could easily be ported to
HipHop. The issue is that there is a bug in the Zend implementation
which someone at Facebook was hitting. So it was fixed with this
rather hackish curl patch.
The history of ext_curl.cpp on github doesn't go back far enough to
show whether there was ever a version of curl_multi_select() in HipHop
which was based on the Zend version. If there was such a version, you
could use it.
I'm working on a version which will use libcurl as it's meant to be
used and also fix the bug, hopefully making everyone happy.
-- Tim Starling