A dependency has been added in 5.6.0 to ext-pcntl, and it's breaking everything, the extension needs to be added to the docker test image. Why such a change on a minor update ?
This type of changes should go to next major version 6.0 i think.
thanks.
@lk77 was there a fix you found for this?
@leecrosdale i reverted to 5.5 for now due to another issue,
but you need to enable pcntl in php, for example for docker :
RUN docker-php-ext-install pcntl
Ah ok, a couple of devs here are running windows, where PCNTL isn't available.
We ended up doing
composer install --ignore-platform-reqs
which seems to work for now!
yes, we had the same issue for windows.
Honestly i think this should be an optional dependency.
I reopen the issue, i had closed it, because we were able to install pcntl on docker, so it was not a big deal for us, but i realise that this issue is more wide than that.
Hmm, that indeed seems to have broken Windows support. We'll try to reverse it later today.
This was the PR btw: https://github.com/laravel/dusk/pull/682
I sent in a PR here: https://github.com/laravel/dusk/pull/692
If anyone can verify that this works on Windows that'd be great.
@driesvints I can confirm it "works" on Windows == I can install it without --ignore-platform-reqs, and it can run tests.
Most helpful comment
@driesvints I can confirm it "works" on Windows == I can install it without
--ignore-platform-reqs, and it can run tests.