search

php / php-src

The PHP Interpreter
https://www.php.net
Other
37.97k stars 7.73k forks source link

Require `poll()` for UNIX based OSs and improve emulation for Windows #15532

Open Girgias opened 4 weeks ago

Girgias commented 4 weeks ago

Description

This would allow some simplification of the php_network.h header and of our emulation layer.

Initially found while working on #15511

cmb69 commented 4 weeks ago

Makes sense to do this, but unless we are absolutely sure that all non Windows we are supporting support poll(), maybe it's better to postpone that to PHP next (8.5 or 9.0).

Girgias commented 4 weeks ago

This is definitely not urgent, the point of the issue is to have a reminder somewhere of this!

I also think it is better to wait for the next release as we're already in beta, and I am a bit vary to change such an aspect.