Merge pull request #39 from zKoz210/fixed-terminating-php82

Fixed #36
spatie · May 16, 2023 · ad46206 · ad46206
2 parents 575df36 + 4ec939c
commit ad46206
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/README.md b/README.md
@@ -46,6 +46,7 @@ on [our virtual postcard wall](https://spatie.be/open-source/postcards).
 This package requires PHP 8 and the [pcntl](https://www.php.net/manual/en/intro.pcntl.php) extensions which is installed in many Unix and Mac systems by default.
 
 ❗️ [pcntl](https://www.php.net/manual/en/intro.pcntl.php) only works in CLI processes, not in a web context.
+❗️ [posix](https://www.php.net/manual/en/book.posix.php) required for correct handling of process termination for Alpine Linux.
 
 ## Installation
 

diff --git a/src/Fork.php b/src/Fork.php
@@ -127,6 +127,10 @@ protected function forkForTask(Task $task): Task
 
             $this->executeInChildTask($task, $socketToParent);
 
+            if (extension_loaded('posix')) {
+                posix_kill(getmypid(), SIGKILL);
+            }
+
             exit;
         }