background_process 1.2 → 1.3

data/Gemfile

@@ -0,0 +1,2 @@
+source :rubygems
+gemspec

data/Gemfile.lock

@@ -0,0 +1,26 @@
+PATH
+  remote: .
+  specs:
+    background_process (1.3)
+
+GEM
+  remote: http://rubygems.org/
+  specs:
+    diff-lcs (1.1.2)
+    rake (0.8.7)
+    rspec (2.5.0)
+      rspec-core (~> 2.5.0)
+      rspec-expectations (~> 2.5.0)
+      rspec-mocks (~> 2.5.0)
+    rspec-core (2.5.1)
+    rspec-expectations (2.5.0)
+      diff-lcs (~> 1.1.2)
+    rspec-mocks (2.5.0)
+
+PLATFORMS
+  ruby
+
+DEPENDENCIES
+  background_process!
+  rake
+  rspec (= 2.5.0)

data/README.textile

@@ -1,8 +1,8 @@
 h1. Background Process
 
 This is like popen4, but provides several convenience methods for interacting
-with the process. It only works on UNIX and Ruby implementations that support
-fork and native UNIX I/O streams.
+with the process. It only works on POSIX and Ruby implementations that support
+fork and native POSIX I/O streams.
 
 "Click here for Complete Documentation":http://rdoc.info/projects/timcharper/background_process
 
@@ -17,7 +17,7 @@ process.exitstatus # => 1
 </pre>
 
 <pre>
-process = BackgroundProcess.run("sh -c 'sleep 1; exit 1'")
+process = BackgroundProcess.run("sh", "-c", "sleep 1; exit 1")
 process.kill("KILL") # => true
 process.running? # => false
 process.exitstatus # => nil

data/Rakefile

@@ -0,0 +1,10 @@
+require 'rake'
+require 'spec/rake/spectask'
+
+desc "Run all examples"
+Spec::Rake::SpecTask.new('spec') do |t|
+  t.spec_files = FileList['spec/**/*.rb']
+end
+
+desc "Default Task"
+task :default => [ :spec ]

data/lib/background_process.rb

@@ -1,3 +1,4 @@
+require 'timeout'
 require File.dirname(__FILE__) + "/background_process/background_process"
 require File.dirname(__FILE__) + "/background_process/pty_background_process"
 require File.dirname(__FILE__) + "/background_process/io_helpers"

data/lib/background_process/background_process.rb

@@ -21,8 +21,8 @@ class BackgroundProcess
   # If you can't control the program and have it explicitly flush its output when it
   # should, or you can't tell the streams to run in sync mode, see
   # PTYBackgroundProcess.run for a workaround.
-  def self.run(command)
-    command = sanitize_params(command) if command.is_a?(Array)
+  def self.run(*command_with_args)
+    command = sanitize_command(command_with_args)
     child_stdin, parent_stdin = IO::pipe
     parent_stdout, child_stdout = IO::pipe
     parent_stderr, child_stderr = IO::pipe
@@ -99,8 +99,10 @@ class BackgroundProcess
 
   protected
   # It's protected. What do you care? :P
-  def self.sanitize_params(params)
-    params.map { |p| p.gsub(' ', '\ ') }.join(" ")
+  def self.sanitize_command(*args)
+    command_and_args = args.flatten
+    return command_and_args.first if command_and_args.length == 1
+    command_and_args.map { |p| p.gsub(' ', '\ ') }.join(" ")
   end
 
   def select_streams(which)

data/lib/background_process/pty_background_process.rb

@@ -11,7 +11,8 @@ class PTYBackgroundProcess < BackgroundProcess
   # * You can't get the exit status
   # * When the process dies, whatever output you haven't read yet is lost.
   # * stderr is merged into stdout
-  def self.run(command)
+  def self.run(*command_with_args)
+    command = sanitize_command(command_with_args)
     thread = Thread.new do # why run PTY separate thread? When a PTY instance
                            # dies, it raises PTY::ChildExited on the thread that
                            # spawned it, interrupting whatever happens to be

data/spec/background_process/pty_background_process_spec.rb

@@ -18,6 +18,12 @@ describe PTYBackgroundProcess do
       process.stdin.puts "exit"
       process.wait
     end
+
+    it "allows you to pass an array of args, which are automatically sanitized" do
+      process = PTYBackgroundProcess.run("sh", "-c", "echo hi")
+      process.stdout.gets.chomp.should == "hi"
+    end
+
   end
 
   describe "#exitstatus" do

metadata

@@ -1,11 +1,12 @@
 --- !ruby/object:Gem::Specification 
 name: background_process
 version: !ruby/object:Gem::Version 
-  prerelease: false
+  hash: 9
+  prerelease: 
   segments: 
   - 1
-  - 2
-  version: "1.2"
+  - 3
+  version: "1.3"
 platform: ruby
 authors: 
 - Tim Harper
@@ -13,10 +14,39 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-04-28 00:00:00 -06:00
+date: 2011-05-19 00:00:00 -06:00
 default_executable: 
-dependencies: []
-
+dependencies: 
+- !ruby/object:Gem::Dependency 
+  name: rspec
+  prerelease: false
+  requirement: &id001 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - "="
+      - !ruby/object:Gem::Version 
+        hash: 27
+        segments: 
+        - 2
+        - 5
+        - 0
+        version: 2.5.0
+  type: :development
+  version_requirements: *id001
+- !ruby/object:Gem::Dependency 
+  name: rake
+  prerelease: false
+  requirement: &id002 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
+        version: "0"
+  type: :development
+  version_requirements: *id002
 description: A library for spawning and interacting with UNIX processes
 email: 
 - timcharper+bp@gmail.com
@@ -30,6 +60,10 @@ extra_rdoc_files:
 files: 
 - README.textile
 - MIT-LICENSE
+- Gemfile
+- Gemfile.lock
+- Rakefile
+- .gemtest
 - lib/background_process/background_process.rb
 - lib/background_process/io_helpers.rb
 - lib/background_process/pty_background_process.rb
@@ -49,23 +83,27 @@ rdoc_options:
 require_paths: 
 - lib
 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 
+  none: false
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
+      hash: 3
       segments: 
       - 0
       version: "0"
 requirements: []
 
 rubyforge_project: background_process
-rubygems_version: 1.3.6
+rubygems_version: 1.5.0
 signing_key: 
 specification_version: 3
 summary: background_process