Correct parameter naming for PTY.spawn (#678)
authorLeonardo Mosquera <ldmosquera@gmail.com>
Mon, 13 Feb 2023 23:19:47 +0000 (20:19 -0300)
committerGitHub <noreply@github.com>
Mon, 13 Feb 2023 23:19:47 +0000 (20:19 -0300)
Per the docs, first parameter for the block is STDOUT + STDERR, 2nd one is STDIN:
https://ruby-doc.org/3.1.3/exts/pty/PTY.html#method-c-spawn

As it is now, the method looks like it's reading from STDIN, though in
fact it's reading from an incorrectly named parameter for STDOUT.

image/auto_build.rb

index d2f830c32fd3641951a6f0443c73722457409dfa..b507ec6f5885d150eeb39ef73436dc2d720c4768 100644 (file)
@@ -12,9 +12,9 @@ images = {
 
 def run(command)
   lines = []
-  PTY.spawn(command) do |stdin, stdout, pid|
+  PTY.spawn(command) do |stdout, stdin, pid|
     begin
-      stdin.each do |line|
+      stdout.each do |line|
         lines << line
         puts line
       end