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

quit command immediately #819

Merged
merged 1 commit into from
Nov 22, 2022
Merged

quit command immediately #819

merged 1 commit into from
Nov 22, 2022

Conversation

ko1
Copy link
Collaborator

@ko1 ko1 commented Nov 22, 2022

quit command kills Ruby process with exit method on the session
thread. Ruby threads receives SystemExit exception, but it doesn't
receive immediately and runs some lines after quit command.

This patch exits the process immediately.

Note that on the remote debugging quit command doesn't kill the
debuggee process but kills remote client process and debuggee process
continues the running.

quit means kills a debugger process and on the local debugging
debuggee process is on the debugger process, so that killing
debugger process kills debuggee process too.

fix #815

`quit` command kills Ruby process with `exit` method on the session
thread. Ruby threads receives `SystemExit` exception, but it doesn't
receive immediately and runs some lines after `quit` command.

This patch exits the process immediately.

Note that on the remote debugging `quit` command doesn't kill the
debuggee process but kills remote client process and debuggee process
continues the running.

`quit` means kills a debugger process and on the local debugging
debuggee process is on the debugger process, so that killing
debugger process kills debuggee process too.

fix #815
@ko1 ko1 merged commit c44094b into master Nov 22, 2022
@ko1 ko1 deleted the quit_immediately branch November 22, 2022 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

q! command doesn't exit remote debuggee correctly
1 participant