Endless loop outside of tests about fork

[dhrrgn]

PCNTL should only be used in the CLI.

Per the docs:

Process Control should not be enabled within a web server environment and unexpected results may happen if any Process Control functions are used within a web server environment.

The behavior you are experiencing is because the process you are forking is the php-fpm process...which is always going to cause weird things to happen.

[dhrrgn]

@freekmurze May be worth adding a note in the README about this. Maybe even add a check like:

if (php_sapi_name() !== 'cli') {
    throw new RuntimeException('Process forking is only supported in CLI mode');
}

[lukeraymonddowning]

Doh! Sorry about this 🤦🏼‍♂️ Thanks for the info!

[freekmurze]

We should definitely mention in the readme that it doesn't word in a web context.
I'm a bit torn what we should do when this code is executed in a web context: throw an exception like suggested or above, or silently execute all given closures without concurrency.

@brendt What do you think?

[brendt]

Definitely throw an exception :) I'll make the necessary changes.

[brendt]

Done: https://github.com/spatie/fork/releases/tag/1.0.1

[jasonvarga]

Sorry to comment on a closed issue.

function_exists('pcntl_fork') returns true in my browser. (At least, when using Valet)

6c4c56c