This seems sensible. I'll do some testing on Windows.

Some interesting context and alternative approaches in golang/dep#862.

thanks for the merge @lox

Some interesting context and alternative approaches in golang/dep#862.

Very useful reference. Any objections to incorporating TerminateTree() for the cancellation as implemented here?

@amitsaha I didn't see any huge upside in TerminateTree() over shelling out to taskkill, did you?

@lox It looks like TerminateTree would kill the entire tree i.e. including processes spawned by child processes of the parent process? (whereas, it seems taskkill /T would kill only the immediate child processes)? (I haven't verified the taskkill /T behavior, so could be wrong)

Ah right, in that case I'd absolutely support TerminateTree() being used, I assumed the /T would have the same behaviour.

I will have to look into this more, but it seems taskkill /T does also terminate child processes spawned by child processes.

indeed, that's the case, so, we don't need TerminateTree. Some demo code: https://github.com/amitsaha/golang-powershell-task-kill-demo