rb-process 0.2.3 → 0.2.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0c6585e505fa625ca039c68f85a1a9ead092158692360ef2a81cb8ac49e9830a
-  data.tar.gz: 8473651b5665ca67caf003acc2837d24e3c7e4eb290e4b343a945df15731901c
+  metadata.gz: 9db5267a9444b8ccfd63444afaf52f0464cede6a9ca0057cbbb7394260dc786b
+  data.tar.gz: 19733804591980297c8abd49ebcab099f4079e5278bb7c79577ccde9f38954c9
 SHA512:
-  metadata.gz: 628904525631dc3e666a8414f9c6da86f6ec615eff3d249fc2f7f5d1fe281e8a35e052326df4d91fc6303d475477d9192189c0811a8a1a7e37d1ba65144664d2
-  data.tar.gz: 2f5f024b53c5fecc60df0d567850332ff1d5f8dd96f5ba3917ad0c58d246fce8b110f95d9fea7314d6df557740cbd50f013cc9604da41bce0271ffde9e67908b
+  metadata.gz: a5cc08c8e200885ad9fb41443f95659af1caad485df71634aace7dc9ba0416034a28f849c9aefeda557c08bb9bb71ae8373a0bb9fe45464891d23a6ada540bee
+  data.tar.gz: b24cc79693d1f1a2aaca5be41896ed15329eb831eeeb7383ef928fe4e80f9e34b88987d10f6093e93a35a623985557ab8cef55671c48e0c9a1e1b4b131b38802

data/.rubocop.yml

@@ -10,8 +10,17 @@ Metrics/BlockLength:
 Metrics/AbcSize:
   Enabled: false
 
+Metrics/CyclomaticComplexity:
+  Enabled: false
+
+Metrics/PerceivedComplexity:
+  Enabled: false
+
 Style/ConditionalAssignment:
   Enabled: false
 
 Style/RaiseArgs:
-  Enabled: false
+  Enabled: false
+
+Style/SpecialGlobalVars:
+  EnforcedStyle: use_builtin_english_names

data/README.md

@@ -17,7 +17,7 @@ If bundler is not being used to manage dependencies, install the gem by executin
 ```ruby
 require "rb/process"
 
-Process.run("bash", "r+") { |pipe| pipe.puts "uname" }
+Process.run("bash") { |pipe| pipe.puts "uname" }
 # => "Linux\n"
 
 Process.output("ls spec").each_line(chomp: true).to_a

data/lib/rb/io/multi_writer.rb

@@ -18,12 +18,11 @@ class IO
     attr_accessor :writers
     attr_reader :closed
 
-    @closed = false
-
     alias_method :sync_close?, :sync_close
     alias_method :closed?, :closed
 
     def initialize(*writers, sync_close: false)
+      @closed = false
       @writers = writers
       @sync_close = sync_close
     end

data/lib/rb/io/stapled.rb

@@ -23,32 +23,25 @@ class IO
     # Underlying `IO`s might have a different status.
     attr_reader :closed
 
-    @closed = false
-
-    WRITER_DELEGATE = %w[
-      <<
-    ].freeze
-
-    READER_DELEGATE = %w[
-      eof
-      eof?
-    ].freeze
+    WRITER_DELEGATE = IO.instance_methods.grep(/^put|^print|write|<</).freeze
+    READER_DELEGATE = IO.instance_methods.grep(/^get[a-z]*|read|each|eof/).freeze
 
     alias_method :sync_close?, :sync_close
     alias_method :closed?, :closed
 
     # Creates a new `IO::Stapled` which reads from *reader* and writes to *writer*.
     def initialize(reader, writer, sync_close: false)
+      @closed = false
       @reader = reader
       @writer = writer
       @sync_close = sync_close
     end
 
     def method_missing(name, *args, &block)
-      if write_methods?(name.to_s)
+      if WRITER_DELEGATE.include?(name)
         check_open
         @writer.send(name, *args, &block)
-      elsif read_methods?(name.to_s)
+      elsif READER_DELEGATE.include?(name)
         check_open
         @reader.send(name, *args, &block)
       else
@@ -57,17 +50,7 @@ class IO
     end
 
     def respond_to_missing?(name, include_private = false)
-      super || write_methods?(name.to_s) || read_methods?(name.to_s)
-    end
-
-    def write_methods?(name)
-      name.include?("put") || name.include?("prin") ||
-        name.include?("write") || WRITER_DELEGATE.include?(name)
-    end
-
-    def read_methods?(name)
-      name.include?("get") || name.include?("read") ||
-        name.include?("each") || READER_DELEGATE.include?(name)
+      WRITER_DELEGATE.include?(name) || READER_DELEGATE.include?(name) || super
     end
 
     # Flushes `writer`.

data/lib/rb/process/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Process
-  VERSION = "0.2.3"
+  VERSION = "0.2.4"
 end

data/lib/rb/process.rb

@@ -17,11 +17,11 @@ module Process
     end
 
     def method_missing(name, *args, &block)
-      @status.send(name, *args, &block)
+      @status.respond_to?(name) ? @status.send(name, *args, &block) : super
     end
 
     def respond_to_missing?(name, include_private = false)
-      super || @status.respond_to?(name, include_private)
+      @status.respond_to?(name, include_private) || super
     end
 
     def exit_code
@@ -35,47 +35,47 @@ module Process
     alias_method :ok?, :success?
   end
 
-  def self.run(*args, out: $stdout, err: $stderr, **options)
-    out_r, out_w = IO.pipe
-    err_r, err_w = IO.pipe
-    child_io = [out_w, err_w]
-    parent_io = [out_r, err_r]
+  def self.run(*args, out: $stdout, err: $stderr, exception: false, **options)
+    stdout_reader, stdout_writer  = IO.pipe
+    stderr_reader, stderr_writer  = IO.pipe
+    childs_io = [stdout_writer, stderr_writer]
+    parent_io = [stdout_reader, stderr_reader]
 
-    output_strio = StringIO.new
-    error_strio  = StringIO.new
-    output_writter = IO::MultiWriter.new(out, output_strio)
-    error_writter = IO::MultiWriter.new(err, error_strio)
+    out_strio = StringIO.new
+    err_strio = StringIO.new
+    out_multiwriter = out.is_a?(Array) ? IO::MultiWriter.new(*out, out_strio) : IO::MultiWriter.new(out, out_strio)
+    err_multiwriter = err.is_a?(Array) ? IO::MultiWriter.new(*err, err_strio) : IO::MultiWriter.new(err, err_strio)
 
     if block_given?
-      in_r, in_w = IO.pipe
-      in_w.sync = true
-      child_io << in_r
-      parent_io << in_w
+      stdin_reader, stdin_writer = IO.pipe
+      stdin_writer.sync = true
+      childs_io << stdin_reader
+      parent_io << stdin_writer
 
-      pid = Process.spawn(*args, **options, in: in_r, out: out_w, err: err_w)
-      child_io.each(&:close)
+      pid = Process.spawn(*args, **options, in: stdin_reader, out: stdout_writer, err: stderr_writer)
+      childs_io.each(&:close)
 
-      pipe = IO::Stapled.new(out_r, in_w)
+      pipe = IO::Stapled.new(stdout_reader, stdin_writer)
       begin
         yield pipe
       ensure
-        in_w.close unless in_w.closed?
+        stdin_writer.close unless stdin_writer.closed?
         pipe.close
       end
     else
-      pid = Process.spawn(*args, **options, out: out_w, err: err_w)
-      child_io.each(&:close)
+      pid = Process.spawn(*args, **options, out: stdout_writer, err: stderr_writer)
+      childs_io.each(&:close)
     end
 
     t1 = Thread.new do
-      out_r.each_line do |line|
-        output_writter.write(line)
+      stdout_reader.each_line do |line|
+        out_multiwriter.write(line)
       end
     end
 
     t2 = Thread.new do
-      err_r.each_line do |line|
-        error_writter.write(line)
+      stderr_reader.each_line do |line|
+        err_multiwriter.write(line)
       end
     end
 
@@ -83,21 +83,38 @@ module Process
       t1.join
       t2.join
     ensure
-      output_writter.close
-      error_writter.close
+      out_multiwriter.close
+      err_multiwriter.close
       parent_io.each { |io| io.close unless io.closed? }
     end
 
     pid, status = Process.wait2(pid)
     if status.success?
-      output_strio.string
+      out_strio.string
     else
-      Err.new(output_strio.string, error_strio.string, status)
+      Err.new(out_strio.string, err_strio.string, status)
     end
+  rescue Errno::ENOENT => e
+    raise e if exception
+
+    Err.new(nil, nil, $?)
   end
 
-  def self.output(...)
-    IO.popen(...).read
+  def self.output(*args, **options)
+    stdout_reader, stdout_writer = IO.pipe
+    pid = Process.spawn(*args, **options, out: stdout_writer)
+    stdout_writer.close
+
+    pid, status = Process.wait2(pid)
+    if status.exited? && status.success?
+      stdout_reader.read
+    else
+      nil
+    end
+  rescue Errno::ENOENT
+    nil
+  ensure
+    stdout_reader.close
   end
 
   def self.code(...)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rb-process
 version: !ruby/object:Gem::Version
-  version: 0.2.3
+  version: 0.2.4
 platform: ruby
 authors:
 - initdc