Set stdin in UpstreamProcess to os.Stdin (instead of /dev/null)

[3v0k4]

Fixes #16

As explained in the issue, debug depends on irb, which depends on reline. There's a call to @@input.winsize in reline:

  def self.get_screen_size
    s = @@input.winsize
    return s if s[0] > 0 && s[1] > 0
    s = [ENV["LINES"].to_i, ENV["COLUMNS"].to_i]
    return s if s[0] > 0 && s[1] > 0
    [24, 80]
  rescue Errno::ENOTTY
    [24, 80]
  end

The above fails because thruster uses /dev/null as stdin:

$ irb

STDIN.winsize
 => [39, 172]

devnull = File.open(File::NULL, "w")
devnull.winsize
(irb):3:in `winsize': Operation not supported by device - /dev/null (Errno::ENODEV)

[kevinmcconnell]

Thanks for the fix, @3v0k4! 🙏

I left a small suggestion, let me know what you think. Otherwise this looks great to me.

[kevinmcconnell]

We currently connect stdout and stderr inside Run(). It would be good to handle these all in the same place, for consistency.

Maybe move the stdin case to Run() too, to match the others?

[3v0k4]

I missed that 🤦 Moved it inside Run().

[kevinmcconnell]

Excellent, thanks again for diagnosing and fixing! 🎉