Don't join on the stdin pass-through thread.

Since the thread would otherwise block forever waiting for user input, we'll wrap it in a timeout and loop checking the status of the sub-process STDIN pipe. This should prevent runaway thread growth.

Author: nirvdrum

lib/svn2git/migration.rb

@@ -1,6 +1,7 @@
 require 'optparse'
 require 'pp'
 require 'open4'
+require 'timeout'
 
 module Svn2Git
   DEFAULT_AUTHORS_FILE = "~/.svn2git/authors"
@@ -376,10 +377,12 @@ def run_command(cmd, exit_on_error=true, printout_output=false)
           end
         end
 
-        threads << Thread.new(stdin) do |stdin|
-          user_reply = $stdin.gets.chomp
-          stdin.puts user_reply
-          stdin.close
+        Thread.new(stdin) do |stdin|
+          until $stdin.closed?
+            user_reply = Timeout.timeout(5) { $stdin.gets.chomp }
+            stdin.puts user_reply
+            stdin.close
+          end
         end
 
         threads.each(&:join)