amphp / process

An async process dispatcher for Amp.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

posix/Runner::kill not worked

Ekstazi opened this issue · comments

some example code:

$process = yield Amp\Parallel\Context::run('test.php');
$process->kill();

test.php:

sleep(30);

As i can see on linux systems(ps -aux | grep php) test,php keep runned and detached after end of main script.

This happen because kill only terminate sh but keep php detached.

I'm also affected by this. @kelunik @trowski please take inspiration from symfony/process and have less naive killing logic. Just proc_terminate is not going to cut it.

@ostrolucky I was waiting for @Ekstazi to update the PR to remove the change he made to the travis config… but that didn't happen and I forgot about it. I'll update it myself and merge it.

@trowski Sorry. I didn't find way to fix seg fault yet... Can you help me with it ?