Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Windows detach process #167

Merged
merged 2 commits into from
Jun 17, 2020
Merged

Windows detach process #167

merged 2 commits into from
Jun 17, 2020

Conversation

soapy1
Copy link
Contributor

@soapy1 soapy1 commented Jun 16, 2020

No description provided.

Using the job flag 'JOB_OBJECT_LIMIT_KILL_ON_JOB_CLOSE' will
destroy all related processes once a job is destroyed.
ref: https://docs.microsoft.com/en-us/windows/win32/procthread/job-objects#managing-job-objects
If a process is created with the 'CREATE_BREAKAWAY_FROM_JOB' flag
then the job must also set the 'JOB_OBJECT_LIMIT_BREAKAWAY' flag.

ref: https://docs.microsoft.com/en-us/windows/win32/procthread/process-creation-flags
@coveralls
Copy link

coveralls commented Jun 16, 2020

Coverage Status

Coverage remained the same at 90.348% when pulling 96b6b09 on soapy1:windows-detach-process into 8e809fe on enkessler:master.

@soapy1 soapy1 marked this pull request as draft June 16, 2020 16:41
@soapy1 soapy1 marked this pull request as ready for review June 16, 2020 21:57
@soapy1
Copy link
Contributor Author

soapy1 commented Jun 16, 2020

Looks like there are similar failures on master 😭

@sds sds added the bug label Jun 17, 2020
@sds sds merged commit a8302c6 into enkessler:master Jun 17, 2020
@sds
Copy link
Collaborator

sds commented Jun 17, 2020

Thanks for the PR! The CI failures seem completely unrelated to this PR. Given Ruby 2.3 is EOL, I will remove from our build matrix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants