cocaine 0.5.1 → 0.5.2

data/.gitignore

@@ -5,3 +5,4 @@ Gemfile.lock
 *.swp
 *.swo
 .bundle
+bin

data/.travis.yml

@@ -1,5 +1,8 @@
 rvm:
-  - 1.8.7
-  - 1.9.2
   - 1.9.3
-  - jruby
+  - 2.0.0
+  - jruby-19mode
+
+matrix:
+  allow_failures:
+    - rvm: rbx-19mode

data/README.md

@@ -34,6 +34,15 @@ line = Cocaine::CommandLine.new("cat", ":file")
 line.command(:file => "ohyeah?'`rm -rf /`.ha!") # => "cat 'ohyeah?'\\''`rm -rf /`.ha!'"
 ```
 
+NOTE: It only does that for arguments interpolated via `run`, NOT argumnets
+passed into `new` (see 'Security' below):
+
+```ruby
+line = Cocaine::CommandLine.new("echo", "haha`whoami`")
+line.command # => "echo haha`whoami`"
+line.run # => "hahawebserver"
+```
+
 You can ignore the result:
 
 ```ruby
@@ -116,6 +125,17 @@ Cocaine::CommandLine.logger = Logger.new(STDOUT)
 Cocaine::CommandLine.new("date").run # => Logs this -> Command :: date
 ```
 
+## Security
+
+Short version: Only pass user-generated data into the `run` method and NOT
+`new`.
+
+As shown in examples above, Cocaine will only shell-escape what is passed in as
+interpolations to the `run` method. It WILL NOT escape what is passed in to the
+second argument of `new`. Cocaine assumes that you will not be manually
+passing user-generated data to that argument and will be using it as a template
+for your command line's structure.
+
 ## POSIX Spawn
 
 You can potentially increase performance by installing [the posix-spawn
@@ -141,7 +161,9 @@ Cocaine::CommandLine.runner = Cocaine::CommandLine::BackticksRunner.new
 And if you really want to, you can define your own Runner, though I can't
 imagine why you would.
 
-### JRuby Caveat
+### JRuby issues
+
+#### Caveat
 
 If you get `Error::ECHILD` errors and are using JRuby, there is a very good
 chance that the error is actually in JRuby. This was brought to our attention
@@ -149,6 +171,16 @@ in https://github.com/thoughtbot/cocaine/issues/24 and probably fixed in
 http://jira.codehaus.org/browse/JRUBY-6162. You *will* want to use the
 `BackticksRunner` if you are unable to update JRuby.
 
+#### Spawn warning
+
+If you get `unsupported spawn option: out` warning (like in [issue 38](https://github.com/thoughtbot/cocaine/issues/38)),
+try to use `PopenRunner`:
+
+```ruby
+Cocaine::CommandLine.runner = Cocaine::CommandLine::PopenRunner.new
+```
+
+
 ## REE
 
 So, here's the thing about REE: The specs that involve timeouts don't work
@@ -171,6 +203,16 @@ Question? Idea? Problem? Bug? Comment? Concern? Like using question marks?
 
 [GitHub Issues For All!](https://github.com/thoughtbot/cocaine/issues)
 
+## Credits
+
+Thank you to all [the contributors](https://github.com/thoughtbot/cocaine/graphs/contributors)!
+
+![thoughtbot](http://thoughtbot.com/logo.png)
+
+Cocaine is maintained and funded by [thoughtbot, inc](http://thoughtbot.com/community)
+
+The names and logos for thoughtbot are trademarks of thoughtbot, inc.
+
 ## License
 
 Copyright 2011-2013 Jon Yurek and thoughtbot, inc. This is free software, and

data/cocaine.gemspec

@@ -21,7 +21,7 @@ Gem::Specification.new do |s|
   s.add_development_dependency('bourne')
   s.add_development_dependency('mocha')
   s.add_development_dependency('rake')
-  s.add_development_dependency('active_support')
+  s.add_development_dependency('activesupport', ">= 3.0.0", "< 5.0")
   s.add_development_dependency('pry')
 end
 

data/lib/cocaine/command_line.rb

@@ -15,15 +15,6 @@ module Cocaine
         @supplemental_environment['PATH'] = (Array(supplemental_path) + [ENV['PATH']]).join(File::PATH_SEPARATOR)
       end
 
-      def posix_spawn_available?
-        @posix_spawn_available ||= begin
-          require 'posix/spawn'
-          true
-        rescue LoadError => e
-          false
-        end
-      end
-
       def environment
         @supplemental_environment ||= {}
       end
@@ -40,12 +31,16 @@ module Cocaine
         @runner = nil
       end
 
+      def java?
+        RUBY_PLATFORM =~ /java/
+      end
+
       private
 
       def best_runner
-        return PosixRunner.new   if posix_spawn_available?
-        return ProcessRunner.new if Process.respond_to?(:spawn)
-        BackticksRunner.new
+        [PosixRunner, ProcessRunner, BackticksRunner].detect do |runner|
+          runner.supported?
+        end.new
       end
     end
 
@@ -74,6 +69,7 @@ module Cocaine
 
     def run(interpolations = {})
       output = ''
+      @exit_status = nil
       begin
         full_command = command(interpolations)
         log("#{colored("Command")} :: #{full_command}")
@@ -81,14 +77,14 @@ module Cocaine
       rescue Errno::ENOENT
         raise Cocaine::CommandNotFoundError
       ensure
-        @exit_status = $?.exitstatus
+        @exit_status = $?.exitstatus if $?.respond_to?(:exitstatus)
       end
-      if $?.exitstatus == 127
+      if @exit_status == 127
         raise Cocaine::CommandNotFoundError
       end
-      unless @expected_outcodes.include?($?.exitstatus)
+      unless @expected_outcodes.include?(@exit_status)
         message = [
-          "Command '#{full_command}' returned #{$?.exitstatus}. Expected #{@expected_outcodes.join(", ")}",
+          "Command '#{full_command}' returned #{@exit_status}. Expected #{@expected_outcodes.join(", ")}",
           "Here is the command output:\n",
           output
         ].join("\n")

data/lib/cocaine/command_line/runners.rb

@@ -3,4 +3,5 @@
 require 'cocaine/command_line/runners/backticks_runner'
 require 'cocaine/command_line/runners/process_runner'
 require 'cocaine/command_line/runners/posix_runner'
+require 'cocaine/command_line/runners/popen_runner'
 require 'cocaine/command_line/runners/fake_runner'

data/lib/cocaine/command_line/runners/backticks_runner.rb

@@ -5,6 +5,13 @@ require 'climate_control'
 module Cocaine
   class CommandLine
     class BackticksRunner
+      def self.supported?
+        true
+      end
+
+      def supported?
+        self.class.supported?
+      end
 
       def call(command, env = {})
         with_modified_environment(env) do

data/lib/cocaine/command_line/runners/fake_runner.rb

@@ -3,6 +3,13 @@
 module Cocaine
   class CommandLine
     class FakeRunner
+      def self.supported?
+        false
+      end
+
+      def supported?
+        self.class.supported?
+      end
 
       attr_reader :commands
 

data/lib/cocaine/command_line/runners/popen_runner.rb

@@ -0,0 +1,29 @@
+# coding: UTF-8
+
+module Cocaine
+  class CommandLine
+    class PopenRunner
+      def self.supported?
+        true
+      end
+
+      def supported?
+        self.class.supported?
+      end
+
+      def call(command, env = {})
+        with_modified_environment(env) do
+          IO.popen(command, "r") do |pipe|
+            pipe.read
+          end
+        end
+      end
+
+      private
+
+      def with_modified_environment(env, &block)
+        ClimateControl.modify(env, &block)
+      end
+    end
+  end
+end

data/lib/cocaine/command_line/runners/posix_runner.rb

@@ -3,28 +3,46 @@
 module Cocaine
   class CommandLine
     class PosixRunner
-      if Cocaine::CommandLine.posix_spawn_available?
-
-        def call(command, env = {})
-          input, output = IO.pipe
-          pid = spawn(env, command, :out => output)
-          output.close
-          result = input.read
-          waitpid(pid)
-          result
+      def self.available?
+        begin
+          require 'posix/spawn'
+          true
+        rescue LoadError => e
+          false
         end
+      end
 
-        private
+      def self.supported?
+        available? && !Cocaine::CommandLine.java?
+      end
 
-        def spawn(*args)
-          POSIX::Spawn.spawn(*args)
-        end
+      def supported?
+        self.class.supported?
+      end
 
-        def waitpid(pid)
-          Process.waitpid(pid)
+      def call(command, env = {})
+        input, output = IO.pipe
+        pid = spawn(env, command, :out => output)
+        output.close
+        result = ""
+        while partial_result = input.read(8192)
+          result << partial_result
         end
+        waitpid(pid)
+        input.close
+        result
+      end
 
+      private
+
+      def spawn(*args)
+        POSIX::Spawn.spawn(*args)
+      end
+
+      def waitpid(pid)
+        Process.waitpid(pid)
       end
+
     end
   end
 end

data/lib/cocaine/command_line/runners/process_runner.rb

@@ -3,29 +3,42 @@
 module Cocaine
   class CommandLine
     class ProcessRunner
-      if Process.respond_to?(:spawn)
-
-        def call(command, env = {})
-          input, output = IO.pipe
-          pid = spawn(env, command, :out => output)
-          output.close
-          result = input.read
-          waitpid(pid)
-          result
-        end
+      def self.available?
+        Process.respond_to?(:spawn)
+      end
 
-        private
+      def self.supported?
+        available? && !Cocaine::CommandLine.java?
+      end
 
-        def spawn(*args)
-          Process.spawn(*args)
-        end
+      def supported?
+        self.class.supported?
+      end
+
+      def call(command, env = {})
+        input, output = IO.pipe
+        pid = spawn(env, command, :out => output)
+        output.close
+        result = input.read
+        waitpid(pid)
+        input.close
+        result
+      end
 
-        def waitpid(pid)
+      private
+
+      def spawn(*args)
+        Process.spawn(*args)
+      end
+
+      def waitpid(pid)
+        begin
           Process.waitpid(pid)
+        rescue Errno::ECHILD => e
+          # In JRuby, waiting on a finished pid raises.
         end
-
       end
+
     end
   end
 end
-

data/lib/cocaine/exceptions.rb

@@ -1,8 +1,8 @@
 # coding: UTF-8
 
 module Cocaine
-  class CommandLineError     < StandardError; end
+  class CommandLineError < StandardError; end
   class CommandNotFoundError < CommandLineError; end
-  class ExitStatusError      < CommandLineError; end
-  class InterpolationError   < CommandLineError; end
+  class ExitStatusError < CommandLineError; end
+  class InterpolationError < CommandLineError; end
 end

data/lib/cocaine/version.rb

@@ -1,5 +1,5 @@
 # coding: UTF-8
 
 module Cocaine
-  VERSION = "0.5.1".freeze
+  VERSION = "0.5.2".freeze
 end

data/spec/cocaine/command_line/runners/backticks_runner_spec.rb

@@ -1,20 +1,22 @@
 require 'spec_helper'
 
 describe Cocaine::CommandLine::BackticksRunner do
-  it_behaves_like 'a command that does not block'
+  if Cocaine::CommandLine::BackticksRunner.supported?
+    it_behaves_like 'a command that does not block'
 
-  it 'runs the command given' do
-    subject.call("echo hello").should == "hello\n"
-  end
+    it 'runs the command given' do
+      subject.call("echo hello").should == "hello\n"
+    end
 
-  it 'modifies the environment and runs the command given' do
-    subject.call("echo $yes", {"yes" => "no"}).should == "no\n"
-  end
+    it 'modifies the environment and runs the command given' do
+      subject.call("echo $yes", {"yes" => "no"}).should == "no\n"
+    end
 
-  it 'sets the exitstatus when a command completes' do
-    subject.call("ruby -e 'exit 0'")
-    $?.exitstatus.should == 0
-    subject.call("ruby -e 'exit 5'")
-    $?.exitstatus.should == 5
+    it 'sets the exitstatus when a command completes' do
+      subject.call("ruby -e 'exit 0'")
+      $?.exitstatus.should == 0
+      subject.call("ruby -e 'exit 5'")
+      $?.exitstatus.should == 5
+    end
   end
 end

data/spec/cocaine/command_line/runners/popen_runner_spec.rb

@@ -0,0 +1,22 @@
+require 'spec_helper'
+
+describe Cocaine::CommandLine::PopenRunner do
+  if Cocaine::CommandLine::PopenRunner.supported?
+    it_behaves_like 'a command that does not block'
+
+    it 'runs the command given' do
+      subject.call("echo hello").should == "hello\n"
+    end
+
+    it 'modifies the environment and runs the command given' do
+      subject.call("echo $yes", {"yes" => "no"}).should == "no\n"
+    end
+
+    it 'sets the exitstatus when a command completes' do
+      subject.call("ruby -e 'exit 0'")
+      $?.exitstatus.should == 0
+      subject.call("ruby -e 'exit 5'")
+      $?.exitstatus.should == 5
+    end
+  end
+end

data/spec/cocaine/command_line/runners/posix_runner_spec.rb

@@ -1,7 +1,7 @@
 require 'spec_helper'
 
 describe Cocaine::CommandLine::PosixRunner do
-  if Cocaine::CommandLine::posix_spawn_available?
+  if Cocaine::CommandLine::PosixRunner.supported?
     it_behaves_like 'a command that does not block'
 
     it 'runs the command given' do
@@ -20,4 +20,3 @@ describe Cocaine::CommandLine::PosixRunner do
     end
   end
 end
-

data/spec/cocaine/command_line/runners/process_runner_spec.rb

@@ -1,7 +1,7 @@
 require 'spec_helper'
 
 describe Cocaine::CommandLine::ProcessRunner do
-  if Process.respond_to?(:spawn)
+  if Cocaine::CommandLine::ProcessRunner.supported?
     it_behaves_like "a command that does not block"
 
     it 'runs the command given' do

data/spec/cocaine/command_line_spec.rb

@@ -69,6 +69,15 @@ describe Cocaine::CommandLine do
     command.run(:hello_world => "Hello, world").should match(/Hello, world/)
   end
 
+  it "does not blow up if running the command errored before the actual execution" do
+    assuming_no_processes_have_been_run
+    command = Cocaine::CommandLine.new("echo", ":hello_world")
+    command.stubs(:command).raises("An Error")
+
+    lambda{ command.run }.should raise_error("An Error")
+    command.exit_status.should be_nil
+  end
+
   it "quotes command line options differently if we're on windows" do
     on_windows!
     cmd = Cocaine::CommandLine.new("convert",
@@ -202,7 +211,7 @@ describe Cocaine::CommandLine do
 
   it 'can still take something that does not respond to tty as a logger' do
     output_buffer = StringIO.new
-    logger = ActiveSupport::BufferedLogger.new(output_buffer)
+    logger = best_logger.new(output_buffer)
     logger.should_not respond_to(:tty?)
     Cocaine::CommandLine.new("echo", "'Logging!' :foo", :logger => logger).run(:foo => "bar")
     output_buffer.rewind
@@ -229,38 +238,39 @@ describe Cocaine::CommandLine do
 
   describe "command execution" do
     it "uses the BackticksRunner by default" do
-      Process.stubs(:respond_to?).with(:spawn).returns(false)
-      Cocaine::CommandLine.stubs(:posix_spawn_available?).returns(false)
+      Cocaine::CommandLine::ProcessRunner.stubs(:supported?).returns(false)
+      Cocaine::CommandLine::PosixRunner.stubs(:supported?).returns(false)
 
       cmd = Cocaine::CommandLine.new("echo", "hello")
+
       cmd.runner.class.should == Cocaine::CommandLine::BackticksRunner
     end
 
     it "uses the ProcessRunner on 1.9 and it's available" do
-      Process.stubs(:respond_to?).with(:spawn).returns(true)
-      Cocaine::CommandLine.stubs(:posix_spawn_available?).returns(false)
+      Cocaine::CommandLine::ProcessRunner.stubs(:supported?).returns(true)
+      Cocaine::CommandLine::PosixRunner.stubs(:supported?).returns(false)
 
       cmd = Cocaine::CommandLine.new("echo", "hello")
       cmd.runner.class.should == Cocaine::CommandLine::ProcessRunner
     end
 
-    it "uses the PosixRunner if the posix-spawn gem is available" do
-      Cocaine::CommandLine.stubs(:posix_spawn_available?).returns(true)
+    it "uses the PosixRunner if the PosixRunner is available" do
+      Cocaine::CommandLine::PosixRunner.stubs(:supported?).returns(true)
 
       cmd = Cocaine::CommandLine.new("echo", "hello")
       cmd.runner.class.should == Cocaine::CommandLine::PosixRunner
     end
 
-    it "uses the BackticksRunner if the posix-spawn gem is available, but we told it to use Backticks all the time" do
-      Cocaine::CommandLine.stubs(:posix_spawn_available?).returns(true)
+    it "uses the BackticksRunner if the PosixRunner is available, but we told it to use Backticks all the time" do
+      Cocaine::CommandLine::PosixRunner.stubs(:supported?).returns(true)
       Cocaine::CommandLine.runner = Cocaine::CommandLine::BackticksRunner.new
 
       cmd = Cocaine::CommandLine.new("echo", "hello")
       cmd.runner.class.should == Cocaine::CommandLine::BackticksRunner
     end
 
-    it "uses the BackticksRunner if the posix-spawn gem is available, but we told it to use Backticks" do
-      Cocaine::CommandLine.stubs(:posix_spawn_available?).returns(true)
+    it "uses the BackticksRunner if the PosixRunner is available, but we told it to use Backticks" do
+      Cocaine::CommandLine::PosixRunner.stubs(:supported?).returns(true)
 
       cmd = Cocaine::CommandLine.new("echo", "hello", :runner => Cocaine::CommandLine::BackticksRunner.new)
       cmd.runner.class.should == Cocaine::CommandLine::BackticksRunner

data/spec/spec_helper.rb

@@ -1,5 +1,5 @@
 require 'rspec'
-require 'mocha'
+require 'mocha/api'
 require 'bourne'
 require 'cocaine'
 require 'timeout'
@@ -14,4 +14,15 @@ RSpec.configure do |config|
   config.mock_with :mocha
   config.include WithExitstatus
   config.include StubOS
+  config.include UnsettingExitstatus
+end
+
+def best_logger
+  if ActiveSupport.const_defined?("Logger")
+    ActiveSupport::Logger
+  elsif ActiveSupport.const_defined?("BufferedLogger")
+    ActiveSupport::BufferedLogger
+  else
+    Logger
+  end
 end

data/spec/support/nonblocking_examples.rb

@@ -8,5 +8,7 @@ shared_examples_for "a command that does not block" do
       $?.exitstatus.should == 0
       output.length.should == 10 * 1024 * 1024
     end
+
+    garbage_file.close
   end
 end

data/spec/support/unsetting_exitstatus.rb

@@ -0,0 +1,7 @@
+module UnsettingExitstatus
+  def assuming_no_processes_have_been_run
+    class << $?
+      undef_method :exitstatus
+    end
+  end
+end

data/spec/support/with_exitstatus.rb

@@ -1,6 +1,6 @@
 module WithExitstatus
   def with_exitstatus_returning(code)
-    saved_exitstatus = $?.nil? ? 0 : $?.exitstatus
+    saved_exitstatus = $?.respond_to?(:exitstatus) ? $?.exitstatus : 0
     begin
       `ruby -e "exit #{code.to_i}"`
       yield

metadata

@@ -1,139 +1,146 @@
---- !ruby/object:Gem::Specification 
+--- !ruby/object:Gem::Specification
 name: cocaine
-version: !ruby/object:Gem::Version 
-  hash: 9
+version: !ruby/object:Gem::Version
+  version: 0.5.2
   prerelease: 
-  segments: 
-  - 0
-  - 5
-  - 1
-  version: 0.5.1
 platform: ruby
-authors: 
+authors:
 - Jon Yurek
 autorequire: 
 bindir: bin
 cert_chain: []
-
-date: 2013-01-12 00:00:00 -05:00
-default_executable: 
-dependencies: 
-- !ruby/object:Gem::Dependency 
+date: 2013-09-27 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
   name: climate_control
-  prerelease: false
-  requirement: &id001 !ruby/object:Gem::Requirement 
+  requirement: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 25
-        segments: 
-        - 0
-        - 0
-        - 3
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
         version: 0.0.3
     - - <
-      - !ruby/object:Gem::Version 
-        hash: 15
-        segments: 
-        - 1
-        - 0
-        version: "1.0"
+      - !ruby/object:Gem::Version
+        version: '1.0'
   type: :runtime
-  version_requirements: *id001
-- !ruby/object:Gem::Dependency 
-  name: rspec
   prerelease: false
-  requirement: &id002 !ruby/object:Gem::Requirement 
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: 0.0.3
+    - - <
+      - !ruby/object:Gem::Version
+        version: '1.0'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-  version_requirements: *id002
-- !ruby/object:Gem::Dependency 
-  name: bourne
   prerelease: false
-  requirement: &id003 !ruby/object:Gem::Requirement 
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: bourne
+  requirement: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-  version_requirements: *id003
-- !ruby/object:Gem::Dependency 
-  name: mocha
   prerelease: false
-  requirement: &id004 !ruby/object:Gem::Requirement 
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: mocha
+  requirement: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-  version_requirements: *id004
-- !ruby/object:Gem::Dependency 
-  name: rake
   prerelease: false
-  requirement: &id005 !ruby/object:Gem::Requirement 
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-  version_requirements: *id005
-- !ruby/object:Gem::Dependency 
-  name: active_support
   prerelease: false
-  requirement: &id006 !ruby/object:Gem::Requirement 
+  version_requirements: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: activesupport
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: 3.0.0
+    - - <
+      - !ruby/object:Gem::Version
+        version: '5.0'
   type: :development
-  version_requirements: *id006
-- !ruby/object:Gem::Dependency 
-  name: pry
   prerelease: false
-  requirement: &id007 !ruby/object:Gem::Requirement 
+  version_requirements: !ruby/object:Gem::Requirement
     none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: 3.0.0
+    - - <
+      - !ruby/object:Gem::Version
+        version: '5.0'
+- !ruby/object:Gem::Dependency
+  name: pry
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-  version_requirements: *id007
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 description: A small library for doing (command) lines
 email: jyurek@thoughtbot.com
 executables: []
-
 extensions: []
-
 extra_rdoc_files: []
-
-files: 
+files:
 - .gitignore
 - .travis.yml
 - GOALS
@@ -148,12 +155,14 @@ files:
 - lib/cocaine/command_line/runners.rb
 - lib/cocaine/command_line/runners/backticks_runner.rb
 - lib/cocaine/command_line/runners/fake_runner.rb
+- lib/cocaine/command_line/runners/popen_runner.rb
 - lib/cocaine/command_line/runners/posix_runner.rb
 - lib/cocaine/command_line/runners/process_runner.rb
 - lib/cocaine/exceptions.rb
 - lib/cocaine/version.rb
 - spec/cocaine/command_line/runners/backticks_runner_spec.rb
 - spec/cocaine/command_line/runners/fake_runner_spec.rb
+- spec/cocaine/command_line/runners/popen_runner_spec.rb
 - spec/cocaine/command_line/runners/posix_runner_spec.rb
 - spec/cocaine/command_line/runners/process_runner_spec.rb
 - spec/cocaine/command_line_spec.rb
@@ -161,44 +170,36 @@ files:
 - spec/support/fake_logger.rb
 - spec/support/nonblocking_examples.rb
 - spec/support/stub_os.rb
+- spec/support/unsetting_exitstatus.rb
 - spec/support/with_exitstatus.rb
-has_rdoc: true
 homepage: http://github.com/thoughtbot/cocaine
 licenses: []
-
 post_install_message: 
 rdoc_options: []
-
-require_paths: 
+require_paths:
 - lib
-required_ruby_version: !ruby/object:Gem::Requirement 
+required_ruby_version: !ruby/object:Gem::Requirement
   none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
-      version: "0"
-required_rubygems_version: !ruby/object:Gem::Requirement 
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
-      version: "0"
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
 requirements: []
-
 rubyforge_project: 
-rubygems_version: 1.6.2
+rubygems_version: 1.8.23
 signing_key: 
 specification_version: 3
 summary: A small library for doing (command) lines
-test_files: 
+test_files:
 - spec/cocaine/command_line/runners/backticks_runner_spec.rb
 - spec/cocaine/command_line/runners/fake_runner_spec.rb
+- spec/cocaine/command_line/runners/popen_runner_spec.rb
 - spec/cocaine/command_line/runners/posix_runner_spec.rb
 - spec/cocaine/command_line/runners/process_runner_spec.rb
 - spec/cocaine/command_line_spec.rb
@@ -206,4 +207,5 @@ test_files:
 - spec/support/fake_logger.rb
 - spec/support/nonblocking_examples.rb
 - spec/support/stub_os.rb
+- spec/support/unsetting_exitstatus.rb
 - spec/support/with_exitstatus.rb