-
Notifications
You must be signed in to change notification settings - Fork 11k
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
[5.3]PHP have to --enable-pcntl if use queue #14909
Comments
Same... Tried to run my queue with
and even
Getting same error... Documentation said nothing about this change, So we must either run everything under Homestead or downgrade to 5.2... @taylorotwell What about windows users? Now we cannot use the queue system. |
Windows users need to pass |
Gah! changing --timeout=0 to --timeout 0 did the trick thanks, it works now |
@GrahamCampbell @taylorotwell we should maybe at a note on the docs regarding this issue. |
Or we could just update the exception message to say turn the timeout off if you don't want timeouts? |
Yeah might be better. |
@gholol - I seem to be getting the same message using either syntax: This is on an 5.2-5.3 upgraded project - I'll try a fresh 5.3 project too. EDIT: Yep - same issue. No error with |
@wdmtech It doesn't work with queue listener, you have to use worker.
|
Thanks @gholol. I got it working that way. Interesting, any idea why not? It has |
Yes looks like we just need to handle fix this on the listen command maybe @GrahamCampbell? Can someone send over a PR? |
I didn't forget about it as such. I just didn't expect anyone to want to use it. |
I mean, we can fix it. :P |
I tried this under my windows with PHP 7.0 (no way to enable pcntl):
php artisan queue:work --timeout=360 --queue=high,low --sleep=3 --tries=3
Will get this exception.
But same project in my server ( Ubuntu 16.04 with PHP 7.0 enable pcntl), it works fine.
So if it is necessary to add in "Server Requirements" ?
The text was updated successfully, but these errors were encountered: