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