Skip to content

[stable23] Improve warning about missing pnctl #32604

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.

Sign up for GitHub

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

Merged
blizzz merged 2 commits into from
Jun 12, 2022
Merged

[stable23] Improve warning about missing pnctl #32604

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
c00ed63
Improve warning about missing pnctl
CarlSchwan May 24, 2022
ce5af32
fix typo
blizzz Jun 10, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion console.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -84,8 +84,9 @@ function exceptionHandler($exception) {
exit(1);
}

if (!function_exists('pcntl_signal') && !in_array('--no-warnings', $argv)) {
if (!(function_exists('pcntl_signal') && function_exists('pcntl_signal_dispatch')) && !in_array('--no-warnings', $argv)) {
echo "The process control (PCNTL) extensions are required in case you want to interrupt long running commands - see https://www.php.net/manual/en/book.pcntl.php" . PHP_EOL;
echo "Additionally the function 'pcntl_signal' and 'pcntl_signal_dispatch' need to be enabeld in your php.ini." . PHP_EOL;
}

$application = new Application(
Expand Down