Use Open3.capture3 instead of IO.popen

jcouball · jcouball · commit d029e4c8995b · 2023-12-26T18:18:18.000-08:00
Signed-off-by: James Couball &lt;jcouball@yahoo.com&gt;
diff --git a/lib/git/base.rb b/lib/git/base.rb
@@ -72,24 +72,25 @@ def self.root_of_worktree(working_dir)
 
       # Option 1 using IO.popen
       #
-      # IO.popen(git_cmd, :chdir => working_dir) do |io|
+      # IO.popen(git_cmd, chdir: working_dir) do |io|
       #   status = Process.wait2(io.pid).last
       #   result = io.read.chomp
       # end
 
       # Option 2 using Open3.popen2
       #
-      Open3.popen2(git_cmd, chdir: working_dir) do |stdin, stdout, wait_thr|
-        status = wait_thr.value
-        result = stdout.chomp
-      end
+      # Open3.popen2(git_cmd, chdir: working_dir) do |stdin, stdout, wait_thr|
+      #   status = wait_thr.value
+      #   result = stdout.read.chomp
+      # end
 
       # Option 3 using Open3.capture3
       #
-      # stdout_s, stderr_s, status = Open3.capture3(custom_git_env_variables, git_cmd, opts)
-      # result = status_s
+      stdout_s, stderr_s, status = Open3.capture3(git_cmd, chdir: working_dir)
+      result = stdout_s
 
       raise ArgumentError, "'#{working_dir}' is not in a git working tree" unless status.success?
+
       result
     end
 
diff --git a/lib/git/lib.rb b/lib/git/lib.rb
@@ -1261,14 +1261,14 @@ def run_command(git_cmd, chdir=nil, &block)
 
       # Option 2 using Open3.popen2
       #
-      Open3.popen2(custom_git_env_variables, git_cmd, opts) do |stdin, stdout, wait_thr|
-        [block.call(stdout), wait_thr.value]
-      end
+      # Open3.popen2(custom_git_env_variables, git_cmd, opts) do |stdin, stdout, wait_thr|
+      #   [block.call(stdout), wait_thr.value]
+      # end
 
       # Option 3 using Open3.capture3
       #
-      # stdout_s, stderr_s, status = Open3.capture3(custom_git_env_variables, git_cmd, opts)
-      # [block.call(StringIO.new(stdout_s)), status]
+      stdout_s, stderr_s, status = Open3.capture3(custom_git_env_variables, git_cmd, opts)
+      [block.call(StringIO.new(stdout_s)), status]
     end
 
     def escape(s)
