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


_______________________________________________
Wikitech-l mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

Reply via email to