notify debugger about gdb_wrapper pid to kill it when debugging finis…

…hed (#149)
ruby-debug · Jun 13, 2018 · bff3618 · bff3618
1 parent 421fb00
commit bff3618
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/bin/gdb_wrapper b/bin/gdb_wrapper
@@ -65,7 +65,10 @@ unless options.ruby_path
   exit 1
 end
 
-argv = '["' + ARGV * '", "' + '"]'
+main_process_argv = ARGV
+main_process_argv << '--gdb-wrapper-pid' << "#{Process.pid}"
+argv = '["' + main_process_argv * '", "' + '"]'
+
 child_argv = '["' + ARGV * '", "' + "', '--ignore-port" + '"]'
 debugger_loader_path = File.expand_path(File.dirname(__FILE__)) + '/../lib/ruby-debug-ide/attach/debugger_loader'
 

diff --git a/bin/rdebug-ide b/bin/rdebug-ide
@@ -28,6 +28,7 @@ options = OpenStruct.new(
     'value_as_nested_element' => false,
     'attach_mode' => false,
     'cli_debug' => false,
+    'gdb_wrapper_pid' => nil,
     'key_value_mode' => false
 )
 
@@ -43,6 +44,8 @@ EOB
 
   opts.on("-h", "--host HOST", "Host name used for remote debugging") {|host| options.host = host}
   opts.on("-p", "--port PORT", Integer, "Port used for remote debugging") {|port| options.port = port}
+  opts.on("-gdbp", "--gdb-wrapper-pid PID", Integer, "Pid of corresponding gdb_wrapper (for attach case)") {|pid| options.gdb_wrapper_pid = pid}
+
   opts.on("--dispatcher-port PORT", Integer, "Port used for multi-process debugging dispatcher") do |dp|
     options.dispatcher_port = dp
   end
@@ -173,6 +176,7 @@ Debugger.value_as_nested_element = options.value_as_nested_element || options.rm
 Debugger.key_value_mode = options.key_value_mode
 
 if options.attach_mode
+  at_exit {Process.kill('INT', options.gdb_wrapper_pid)} if options.gdb_wrapper_pid
   if Debugger::FRONT_END == "debase"
     Debugger.init_variables
   end