childprocess 0.5.0 → 0.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c1bf2633335fa2da24ffd367d4a83e624b9e22b1
-  data.tar.gz: 4f05990d1fb49c76270f9ff11211fe4e05daca46
+  metadata.gz: 0a69d7a6d2d1a65d1496d2b2dc327348cfdf6801
+  data.tar.gz: ec705f40bd0330bea05aa4ed556e047fa8cdf337
 SHA512:
-  metadata.gz: 08bc35f482edba588bf05e54777a2e3f1c2a91c0126a32d9cbdf126511deaf6c6958366e5fafd7e2f00fa370942ffe791fbb1b2aa211a0f34748f94a85789beb
-  data.tar.gz: 7531827c11b12ab23b4f87c487a05bce4cf3fb4a2ee0fd3f45f8dbe7c1803364a7f9577b43c6d0a4b62e92e3bd03ab9a5cb263bf64bf623f1c84a6b7f830c38b
+  metadata.gz: 474cbc9d237b873332e387e602b575628c49479cb60705509fc154dd35b5409fe1b48d2b8b9d5b5775d72b5b3787359cde2b9f35a37fb5834a444efb327b0002
+  data.tar.gz: a25cb4f8d10b38537b101484bebf8888530020ff3876615d4daa5b78ee96b104758973f0e15b2f2e97e50cd82fd5de532d4bff1352a3bcc47d7497dafd0ac745

data/lib/childprocess/unix/posix_spawn_process.rb

@@ -13,18 +13,16 @@ module ChildProcess
         actions = Lib::FileActions.new
         attrs   = Lib::Attrs.new
 
-        if @io
-          if @io.stdout
-            actions.add_dup fileno_for(@io.stdout), fileno_for($stdout)
-          else
-            actions.add_open fileno_for($stdout), "/dev/null", File::WRONLY, 0644
-          end
+        if io.stdout
+          actions.add_dup fileno_for(io.stdout), fileno_for($stdout)
+        else
+          actions.add_open fileno_for($stdout), "/dev/null", File::WRONLY, 0644
+        end
 
-          if @io.stderr
-            actions.add_dup fileno_for(@io.stderr), fileno_for($stderr)
-          else
-            actions.add_open fileno_for($stderr), "/dev/null", File::WRONLY, 0644
-          end
+        if io.stderr
+          actions.add_dup fileno_for(io.stderr), fileno_for($stderr)
+        else
+          actions.add_open fileno_for($stderr), "/dev/null", File::WRONLY, 0644
         end
 
         if duplex?

data/lib/childprocess/version.rb

@@ -1,3 +1,3 @@
 module ChildProcess
-  VERSION = "0.5.0"
+  VERSION = "0.5.1"
 end

data/spec/io_spec.rb

@@ -204,4 +204,15 @@ describe ChildProcess do
       out.close
     end
   end
+
+  it 'should not inherit stdout and stderr by default' do
+    cap = capture_std do
+      process = echo
+      process.start
+      process.wait
+    end
+
+    expect(cap.stdout).to eq ''
+    expect(cap.stderr).to eq ''
+  end
 end

data/spec/spec_helper.rb

@@ -11,6 +11,7 @@ require 'rspec'
 require 'tempfile'
 require 'socket'
 require 'stringio'
+require 'ostruct'
 
 module ChildProcessSpecHelper
   RUBY = defined?(Gem) ? Gem.ruby : 'ruby'
@@ -212,6 +213,26 @@ module ChildProcessSpecHelper
     end
   end
 
+  def capture_std
+    orig_out = STDOUT.clone
+    orig_err = STDERR.clone
+
+    out = Tempfile.new 'captured-stdout'
+    err = Tempfile.new 'captured-stderr'
+    out.sync = true
+    err.sync = true
+
+    STDOUT.reopen out
+    STDERR.reopen err
+
+    yield
+
+    OpenStruct.new stdout: rewind_and_read(out), stderr: rewind_and_read(err)
+  ensure
+    STDOUT.reopen orig_out
+    STDERR.reopen orig_err
+  end
+
 end # ChildProcessSpecHelper
 
 Thread.abort_on_exception = true

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: childprocess
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.5.1
 platform: ruby
 authors:
 - Jari Bakken