Contributor

Hi,

terminateJob is being called from within terminateProcess which already has a lock on the MVar.
This add a new function terminateJobUnsafe which does not take an MVar and so is not thread-safe but can be called from terminateProcess safely.

Fixes #196

Collaborator

The AppVeyor tests failed on this build.

Side note: I think I'll move CI over to Github Actions instead of the Travis/AppVeyor split, just an FYI.

Contributor Author

@snoyberg alright, the test is fixed to work properly on Windows now.

Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Contributor Author

Done.

Collaborator

Thanks!