open4 1.3.3 → 1.3.4

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    YTM2ZTNmNzUxMDkwZWM4YzQ2NjdhOGU5MjdjOGRhY2Y1ODIyMzhkYg==
+    ZTUyNzE3MzczYmM1N2M5ZTQ0MWY0ZTkxMGM2NDcwODQzZWU3YjBkZg==
   data.tar.gz: !binary |-
-    ZTM2NTNmMDEyZWFkMTA2MDM3OTE4YmZjOWE3ZmFiNDE0MGI2OTVkOA==
+    NDU2NWRlNGIxZjg5OGZmYjA5OGZhNDhhMTBiZDI2NTEyYzgxMTc5ZQ==
 !binary "U0hBNTEy":
   metadata.gz: !binary |-
-    MTY0ZDVjOTMyNjU4ZjMwZjIxOGNiNGEzYzQ2MjRjMGRmZDgyYjU1NGIwNTc0
-    MDkwMjU2NjRiYzc4ZjMyOTA3NjAyMmQ5ZjJkYTgzMGQwYWIyZDk1ODBkM2Zj
-    MzUzYjIxYzg5ODdlYzFmMWJiNTc3NjljZWI3ZmNhNzJhNzM5ODI=
+    ZDJjMzYwZjA5OGY5ODI2MDA3YzY5MmVkZjNjYjYzYTQyMjIzNmRjNzk3NDZi
+    NjRiMDExNjZjZDBkMWM5MWY0ZTlmNTYxNDRlODQ5NTNjMjk3YzcyNTYzYzU0
+    ZGIxYTNiZjc0MjdkY2JkZmNkYWIxYWY1MDlmY2RmYTdmZTUyZDk=
   data.tar.gz: !binary |-
-    YWJhZDFkNDUwYTczNDgzOGU1ZjEyMGMwZjg5MDMwOThhYWM5Yjg2NmUwYTkw
-    ZTdlYjc4ZjRhYzM2MjQ5MDBhNDYxZTkzMjgzYzRjZWI3ZjM1MjIxNGQ5OTFl
-    NzNhNjY5NmQzNDczMTg0NzI3YzFmNTg4MmJhNjhkODZjOTI3YzQ=
+    MzkxZDc2ODhjMjk4ZDEwMzM0YjIxYzEyYThiYTg3YmRlOTljYmRiOGUzMGY2
+    NTU0NmM4OTg2OTRlNDYwODMzMmU5ODA5OTkzNmYzMTgxYmIzY2ZlNGQ1NzE4
+    ZmY3ZDNlZmRmYzRiYmE2MGM0MDgxYjJhYWZkZTZhM2Q4ZmIyMzM=

data/lib/open4.rb

@@ -4,7 +4,7 @@ require 'timeout'
 require 'thread'
 
 module Open4
-  VERSION = '1.3.3'
+  VERSION = '1.3.4'
   def Open4.version() VERSION end
 
   def Open4.description
@@ -183,9 +183,9 @@ module Open4
 
       begin
         @argv.each do |a, b|
-          @threads << Thread.new(*a) do |*a|
+          @threads << Thread.new(*a) do |*_a|
             begin
-              b[*a]
+              b[*_a]
             ensure
               killall rescue nil if $!
               @done.push Thread.current
@@ -214,9 +214,9 @@ module Open4
 
   def new_thread *a, &b
     cur = Thread.current
-    Thread.new(*a) do |*a|
+    Thread.new(*a) do |*_a|
       begin
-        b[*a]
+        b[*_a]
       rescue Exception => e
         cur.raise e
       end
@@ -229,8 +229,8 @@ module Open4
       keys, default, _ = args
       catch(:opt) do
         [keys].flatten.each do |key|
-          [key, key.to_s, key.to_s.intern].each do |key|
-            throw :opt, opts[key] if opts.has_key?(key)
+          [key, key.to_s, key.to_s.intern].each do |_key|
+            throw :opt, opts[_key] if opts.has_key?(_key)
           end
         end
         default
@@ -259,19 +259,19 @@ module Open4
         q = Queue.new
         th = nil
 
-        timer_set = lambda do |t|
-          th = new_thread{ to(t){ q.pop } }
+        timer_set = lambda do |_t|
+          th = new_thread{ to(_t){ q.pop } }
         end
 
-        timer_cancel = lambda do |t|
+        timer_cancel = lambda do |_t|
           th.kill if th rescue nil
         end
 
         timer_set[t]
         begin
-          src.each do |buf|
+          src.each do |_buf|
             timer_cancel[t]
-            send_dst[buf]
+            send_dst[_buf]
             timer_set[t]
           end
         ensure
@@ -342,17 +342,17 @@ module Open4
 
               te = ThreadEnsemble.new c
 
-              te.add_thread(i, stdin) do |i, stdin|
-                relay stdin, i, stdin_timeout
-                i.close rescue nil
+              te.add_thread(i, stdin) do |_i, _stdin|
+                relay _stdin, _i, stdin_timeout
+                _i.close rescue nil
               end
 
-              te.add_thread(o, stdout) do |o, stdout|
-                relay o, stdout, stdout_timeout
+              te.add_thread(o, stdout) do |_o, _stdout|
+                relay _o, _stdout, stdout_timeout
               end
 
-              te.add_thread(e, stderr) do |o, stderr|
-                relay e, stderr, stderr_timeout
+              te.add_thread(e, stderr) do |_o, _stderr| # HACK: I think this is a bug
+                relay e, _stderr, stderr_timeout
               end
 
               te.run
@@ -386,7 +386,7 @@ module Open4
       else
         argv.push opts
     end
-    thread = Thread.new(arg, argv){|arg, argv| spawn arg, *argv}
+    thread = Thread.new(arg, argv){|_arg, _argv| spawn _arg, *_argv}
     sc = class << thread; self; end
     sc.module_eval {
       define_method(:pid){ @pid ||= q.pop }

data/open4.gemspec

@@ -3,7 +3,7 @@
 
 Gem::Specification::new do |spec|
   spec.name = "open4"
-  spec.version = "1.3.3"
+  spec.version = "1.3.4"
   spec.platform = Gem::Platform::RUBY
   spec.summary = "open4"
   spec.description = "open child process with handles on pid, stdin, stdout, and stderr: manage child processes and their io handles easily."
@@ -28,11 +28,11 @@ Gem::Specification::new do |spec|
  "samples/stdin_timeout.rb",
  "samples/timeout.rb",
  "test",
+ "test/lib",
+ "test/lib/test_case.rb",
  "test/pfork4_test.rb",
  "test/popen4_test.rb",
  "test/popen4ext_test.rb",
- "test/support",
- "test/support/test_case.rb",
  "white_box",
  "white_box/leak.rb"]
 

data/rakefile

@@ -32,7 +32,7 @@ def run_tests!(which = nil)
 
   test_rbs.each_with_index do |test_rb, index|
     testno = index + 1
-    command = "#{ This.ruby } -w -I ./lib -I ./test/lib #{ test_rb }"
+    command = "#{ This.ruby } -rubygems -w -I ./lib -I ./test/lib #{ test_rb }"
 
     puts
     say(div, :color => :cyan, :bold => true)

data/test/{support → lib}/test_case.rb

@@ -13,7 +13,7 @@ module Open4
     class MyError < RuntimeError; end
 
     def on_mri?
-      ::Config::CONFIG['ruby_install_name'] == 'ruby'
+      ::RbConfig::CONFIG['ruby_install_name'] == 'ruby'
     end
 
     def wait_status(cid)

data/test/pfork4_test.rb

@@ -51,7 +51,7 @@ class PFork4Test < TestCase
     fun = lambda { $stdout.write Process.pid }
     cid_in_block = nil
     cid_in_fun = nil
-    status = pfork4(fun) do |cid, _, stdout, _|
+    pfork4(fun) do |cid, _, stdout, _|
       cid_in_block = cid
       cid_in_fun = stdout.read.to_i
     end

data/test/popen4_test.rb

@@ -8,7 +8,7 @@ class POpen4Test < TestCase
 
   def test_unknown_command_propagates_exception
     err = assert_raises(*UNKNOWN_CMD_ERRORS) { popen4 UNKNOWN_CMD }
-    assert_match /#{UNKNOWN_CMD}/, err.to_s if on_mri?
+    assert_match(/#{UNKNOWN_CMD}/, err.to_s) if on_mri?
   end
 
   def test_exception_propagation_avoids_zombie_child_process
@@ -31,7 +31,7 @@ class POpen4Test < TestCase
     cmd = %{ruby -e "STDOUT.print Process.pid"}
     cid_in_block = nil
     cid_in_fun = nil
-    status = popen4(cmd) do |cid, _, stdout, _|
+    popen4(cmd) do |cid, _, stdout, _|
       cid_in_block = cid
       cid_in_fun = stdout.read.to_i
     end

data/test/popen4ext_test.rb

@@ -9,7 +9,7 @@ class POpen4Test < TestCase
 
   def test_unknown_command_propagates_exception
     err = assert_raises(*UNKNOWN_CMD_ERRORS) { popen4ext true, UNKNOWN_CMD }
-    assert_match /#{UNKNOWN_CMD}/, err.to_s if on_mri?
+    assert_match(/#{UNKNOWN_CMD}/, err.to_s) if on_mri?
   end
 
   def test_exception_propagation_avoids_zombie_child_process
@@ -32,7 +32,7 @@ class POpen4Test < TestCase
     cmd = %{ruby -e "STDOUT.print Process.pid"}
     cid_in_block = nil
     cid_in_fun = nil
-    status = popen4ext(true, cmd) do |cid, _, stdout, _|
+    popen4ext(true, cmd) do |cid, _, stdout, _|
       cid_in_block = cid
       cid_in_fun = stdout.read.to_i
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: open4
 version: !ruby/object:Gem::Version
-  version: 1.3.3
+  version: 1.3.4
 platform: ruby
 authors:
 - Ara T. Howard
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-03-03 00:00:00.000000000 Z
+date: 2014-05-15 00:00:00.000000000 Z
 dependencies: []
 description: ! 'open child process with handles on pid, stdin, stdout, and stderr:
   manage child processes and their io handles easily.'
@@ -32,10 +32,10 @@ files:
 - samples/spawn.rb
 - samples/stdin_timeout.rb
 - samples/timeout.rb
+- test/lib/test_case.rb
 - test/pfork4_test.rb
 - test/popen4_test.rb
 - test/popen4ext_test.rb
-- test/support/test_case.rb
 - white_box/leak.rb
 homepage: https://github.com/ahoward/open4
 licenses: