define lower bound php-version

amphp / process

An async process dispatcher for Amp.

MIT License

233 stars 30 forks source link

define lower bound php-version #1

Closed staabm closed 9 years ago

staabm commented 9 years ago

as discussed for *nix systems 5.4 is enough for win 5.5+ is recommended

bwoebi commented 9 years ago

Is there any way how to specify a Windows specific requirement?

staabm commented 9 years ago

I am not aware of one

kelunik commented 9 years ago

I don't think so. Why is it 5.5 on Windows?

bwoebi commented 9 years ago

@staabm 5.4 is only needed because Amp needs it. If Amp were supporting 5.3, it would work there too on *nix.

I prefer to not add my own requirement here, but let the minimum version of this lib be implicitly defined by Amp.

@kelunik There exist bugs like https://bugs.php.net/bug.php?id=51800 which prevent this lib from working in every case in 5.4 and older… This failure is Windows specific though…