-
-
Notifications
You must be signed in to change notification settings - Fork 726
Implement Eventloop based on pecl/event #177
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
The ticket seems closed :) https://bugs.php.net/bug.php?id=64652 |
I will still need to remove all the php://temp from AbstractLoopTest, since it's not supported yet. |
Is this an @rosmanov statement ? The seg fault is fixed, but no support for in-memory streams ? |
Yep, on the ticket page:
|
Almost all the unit tests error'ed out with:
I installed pecl/event version 1.7.0 via |
Could you provide a sample code snippet? Probably, the fd is a regular file descriptor(not a FIFO, or a pipe thing). |
Doing a real-world test using sockets and it did work better. I did get a |
Again, I'd like to see a simple code snippet, which doesn't require any Framework, for instance. Otherwise it's rather difficult to reproduce and fix. |
Did it work with older versions of "event" ext.? |
This is the only version I have tested. Is there an older version you'd like me to test with? I'll try to put together a raw failing example. |
1.6.2 please http://pecl.php.net/package/event/1.6.2 |
Tested, 1.6.2 also has the same issues. |
Good. It means that I didn't break anything in the recent releases. Waiting for a sample snippet then. |
Closing in favour of #248 |
Fix #161
The tests are failing for now, I submitted an issue about it: https://bugs.php.net/bug.php?id=64652