Remove jruby hack, needs proper solution.

piotrmurach · Apr 16, 2016 · a6919cc · a6919cc
1 parent fbe4f13
commit a6919cc
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 9 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -18,6 +18,8 @@ matrix:
   allow_failures:
     - rvm: ruby-head
     - rvm: jruby-head
+    - rvm: jruby
+    - rvm: jruby-9000
   fast_finish: true
 branches:
   only: master

diff --git a/lib/tty/command/execute.rb b/lib/tty/command/execute.rb
@@ -35,7 +35,7 @@ def spawn(cmd)
           :err => err_wr, err_rd => :close
         }).merge(@process_options)
 
-        pid = _spawn(cmd.to_command, opts)
+        pid = Process.spanw(cmd.to_command, opts)
 
         # close in parent process
         [out_wr, err_wr].each { |fd| fd.close if fd }
@@ -55,14 +55,6 @@ def spawn(cmd)
 
       private
 
-      def jruby?
-        RUBY_PLATFORM =~ /java/
-      end
-
-      def _spawn(cmd, opts)
-        Process.spawn(cmd, jruby? ? {} : opts)
-      end
-
       def normalize_redirect_options(options)
         options.reduce({}) do |opts, (key, value)|
           if fd?(key)