In multi-trigger, if one trigger crashes, others may keep running about spin

[itowlson]

(not you Kate. just the unfeeling cosmos in general)

[itowlson]

Confirmed finalisers do not appear to run.

[kate-goldenring]

@itowlson I was able to repro this. With a neverends plugin:

#!/bin/bash

i=0
while true; do
   echo $i
   i=$(( i+1 ))
   sleep .5
done

Interestingly adding kill_on_drop(true) did resolve it for me. Stopping the spin (cargo run) process also killed the plugin process

    let mut command = Command::new(binary);
    command.args(args);
    command.envs(get_env_vars_map()?);
    command.kill_on_drop(true);

[itowlson]

Oh interesting! How did you stop the Spin process? In my case the parent process is kill_on_drop-ing the Spin process so maybe your way of stopping is giving it a chance to clean up okay. Thanks!

[itowlson]

I had to add a Ctrl+C handler to external - that seemed to work but more cruft sigh

[kate-goldenring]

I cannot repro my success with my changes, so I agree that kill_on_drop does not work. Will also keep trying!

[itowlson]

AHHH THE CTRL+C HANDLER WORKED AND NOW IT DOESN'T

CURSE YOU ALL

[itowlson]

oh SOMETIMES it works