about summary refs log tree commit homepage
path: root/lib/dtas
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2015-01-19 09:17:51 +0000
committerEric Wong <e@80x24.org>2015-01-19 09:58:47 +0000
commitef39866fa6e6ac91cc64f18573dea22c3f75c6e1 (patch)
tree62ab8b9cfdf57acf63b84661f7ed8fa1725b9fc8 /lib/dtas
parentb95a51440b6378847c4ab5dd4cf9e96e18b5f241 (diff)
downloaddtas-ef39866fa6e6ac91cc64f18573dea22c3f75c6e1.tar.gz
Ensure we can apply the workaround to dtas-sourceedit and our
test cases while also simplifying the existing call sites a
little.  This will also make for less code churn in 3-5
years down the line when we drop <= 2.1 support.
Diffstat (limited to 'lib/dtas')
-rw-r--r--lib/dtas/partstats.rb8
-rw-r--r--lib/dtas/process.rb15
-rw-r--r--lib/dtas/spawn_fix.rb8
3 files changed, 13 insertions, 18 deletions
diff --git a/lib/dtas/partstats.rb b/lib/dtas/partstats.rb
index 6c714b3..546b88b 100644
--- a/lib/dtas/partstats.rb
+++ b/lib/dtas/partstats.rb
@@ -10,6 +10,7 @@ require_relative 'sigevent'
 class DTAS::PartStats
   CMD = 'sox "$INFILE" -n $TRIMFX $SOXFX stats $STATSOPTS'
   include DTAS::Process
+  include DTAS::SpawnFix
   attr_reader :key_idx
   attr_reader :key_width
 
@@ -56,12 +57,7 @@ class DTAS::PartStats
     env["INFILE"] = @infile
     env["TRIMFX"] = "trim #{trim_part.tbeg}s #{trim_part.tlen}s"
     opts = { pgroup: true, close_others: true, err: wr }
-    pid = begin
-      Process.spawn(env, CMD, opts)
-    rescue Errno::EINTR
-      # workaround for older Rubies https://bugs.ruby-lang.org/issues/8770
-      retry
-    end
+    pid = spawn(env, CMD, opts)
     wr.close
     [ pid, rd ]
   end
diff --git a/lib/dtas/process.rb b/lib/dtas/process.rb
index c0ce9a3..56e88e8 100644
--- a/lib/dtas/process.rb
+++ b/lib/dtas/process.rb
@@ -7,6 +7,7 @@ require_relative 'xs'
 module DTAS::Process # :nodoc:
   PIDS = {}
   include DTAS::XS
+  include DTAS::SpawnFix
 
   def self.reaper
     begin
@@ -47,12 +48,7 @@ module DTAS::Process # :nodoc:
     opts = { close_others: true, pgroup: true }.merge!(opts)
     env = env_expand(env, opts)
 
-    pid = begin
-      Process.spawn(env, cmd, opts)
-    rescue Errno::EINTR
-      # workaround for older Rubies https://bugs.ruby-lang.org/issues/8770
-      retry
-    end
+    pid = spawn(env, cmd, opts)
     warn [ :spawn, pid, cmd ].inspect if $DEBUG
     @spawn_at = Time.now.to_f
     PIDS[pid] = self
@@ -76,12 +72,7 @@ module DTAS::Process # :nodoc:
       opts[:err] = we
     end
     env = env_expand(env, opts)
-    pid = begin
-      Process.spawn(env, *cmd, opts)
-    rescue Errno::EINTR
-      # workaround for older Rubies https://bugs.ruby-lang.org/issues/8770
-      retry
-    end
+    pid = spawn(env, *cmd, opts)
     w.close
     if err_str
       we.close
diff --git a/lib/dtas/spawn_fix.rb b/lib/dtas/spawn_fix.rb
new file mode 100644
index 0000000..dfcc884
--- /dev/null
+++ b/lib/dtas/spawn_fix.rb
@@ -0,0 +1,8 @@
+module DTAS::SpawnFix # :nodoc:
+  # workaround for older Rubies: https://bugs.ruby-lang.org/issues/8770
+  def spawn(*args)
+    super(*args)
+  rescue Errno::EINTR
+    retry
+  end if RUBY_VERSION.to_f <= 2.1
+end