exceptional_fork 1.0.0 → 1.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 747654a03842e5bf513885263e928420649bc789
-  data.tar.gz: 209e893bffed33cf0610551a228f11c1509580f2
+  metadata.gz: 1c0f186b58e704eeda7fe96b5fe617927fe9e64e
+  data.tar.gz: b44013381bb2431a0d984ac5e9e937385cb65985
 SHA512:
-  metadata.gz: 5266e24496d23f343040a1c7fb8820e4513f6162acea0fb1bc9a4fd8a15e3ce8b23d42741a9801e935ab96c320354f0380a369db91f3783569c14d6549eeb17c
-  data.tar.gz: 1cc0e5f91b46b08dccf7c7870e0551e30dad4eac681af03491e3368ce9484f8c757240061b4a3e52ac45fc587c9497f5aace93441f4ee29d4981121511a350df
+  metadata.gz: 1169303635307088336f99794328231245f870013ecf7eb138fe9dd1fb32f197c85e8df1ad1107c1831120a222cfbd8d38cbbd156c3ed4d48c4fdff34c7046e3
+  data.tar.gz: 15431a14f4ba784cafdb27940709eb03e4280c86c5d06ce01c83e61dae823f7d97a2390114a7b80fe11dd1b31f71a524c477a198e22d12cbe35e248d61f13d48

data/exceptional_fork.gemspec

@@ -0,0 +1,62 @@
+# Generated by jeweler
+# DO NOT EDIT THIS FILE DIRECTLY
+# Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
+# -*- encoding: utf-8 -*-
+# stub: exceptional_fork 1.1.0 ruby lib
+
+Gem::Specification.new do |s|
+  s.name = "exceptional_fork"
+  s.version = "1.1.0"
+
+  s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
+  s.require_paths = ["lib"]
+  s.authors = ["Julik Tarkhanov"]
+  s.date = "2015-12-15"
+  s.description = " Uses pipes to re-raise exceptions. Something better than an exit code has to exist. "
+  s.email = "me@julik.nl"
+  s.extra_rdoc_files = [
+    "LICENSE.txt",
+    "README.rdoc"
+  ]
+  s.files = [
+    ".document",
+    ".rspec",
+    "Gemfile",
+    "LICENSE.txt",
+    "README.rdoc",
+    "Rakefile",
+    "exceptional_fork.gemspec",
+    "lib/exceptional_fork.rb",
+    "spec/exceptional_fork_spec.rb",
+    "spec/spec_helper.rb"
+  ]
+  s.homepage = "http://github.com/julik/exceptional_fork"
+  s.licenses = ["MIT"]
+  s.rubygems_version = "2.2.2"
+  s.summary = "Raise exceptions from the forked child process in the parent"
+
+  if s.respond_to? :specification_version then
+    s.specification_version = 4
+
+    if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
+      s.add_development_dependency(%q<rspec>, ["~> 2.14"])
+      s.add_development_dependency(%q<rdoc>, ["~> 3.12"])
+      s.add_development_dependency(%q<bundler>, ["~> 1.0"])
+      s.add_development_dependency(%q<jeweler>, ["~> 2.0.1"])
+      s.add_development_dependency(%q<simplecov>, [">= 0"])
+    else
+      s.add_dependency(%q<rspec>, ["~> 2.14"])
+      s.add_dependency(%q<rdoc>, ["~> 3.12"])
+      s.add_dependency(%q<bundler>, ["~> 1.0"])
+      s.add_dependency(%q<jeweler>, ["~> 2.0.1"])
+      s.add_dependency(%q<simplecov>, [">= 0"])
+    end
+  else
+    s.add_dependency(%q<rspec>, ["~> 2.14"])
+    s.add_dependency(%q<rdoc>, ["~> 3.12"])
+    s.add_dependency(%q<bundler>, ["~> 1.0"])
+    s.add_dependency(%q<jeweler>, ["~> 2.0.1"])
+    s.add_dependency(%q<simplecov>, [">= 0"])
+  end
+end
+

data/lib/exceptional_fork.rb

@@ -1,5 +1,7 @@
 module ExceptionalFork
-  VERSION = '1.0.0'
+  VERSION = '1.1.0'
+  QUIT = "The child process %d has quit or was killed abruptly. No error information could be retrieved".freeze
+  ProcessHung = Class.new(StandardError)
   
   # Fork with a block and wait until the forked child exits.
   # Any exceptions raised within the block will be re-raised from this
@@ -32,6 +34,10 @@ module ExceptionalFork
     reader.close rescue IOError # Do not leak pipes since the process might be long-lived
   
     if $?.exitstatus != 0 # If the process exited uncleanly capture the error
+      # If the child gets kill -9d then no exception gets written, and no information
+      # gets recovered.
+      raise ProcessHung.new(QUIT % pid) if (child_error.nil? || child_error.empty?)
+      
       unmarshaled_error, backtrace_in_child = Marshal.load(child_error)
       # Pick up the exception 
       reconstructed_error = unmarshaled_error.exception

data/spec/exceptional_fork_spec.rb

@@ -16,4 +16,19 @@ describe "ExceptionalFork" do
       expect($1).not_to eq(pid_of_parent.to_s)
     end
   end
+  
+  it "raises a ProcessHung if no exception information can be recovered" do
+    expect(Process).to receive(:fork).and_call_original
+    
+    pid_of_parent = Process.pid
+    begin
+      Thread.new { sleep 5; `killall -9 ef-test-process` }
+      ExceptionalFork.fork_and_wait { $0 = 'ef-test-process'; sleep 100; }
+      expect(false).to eq(true), "This should never be reached"
+    rescue => e
+      matches = (e.message =~ /No error information could be retrieved/)
+      expect(matches).not_to be_nil
+      expect($1).not_to eq(pid_of_parent.to_s)
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: exceptional_fork
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.1.0
 platform: ruby
 authors:
 - Julik Tarkhanov
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-12-26 00:00:00.000000000 Z
+date: 2015-12-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -95,6 +95,7 @@ files:
 - LICENSE.txt
 - README.rdoc
 - Rakefile
+- exceptional_fork.gemspec
 - lib/exceptional_fork.rb
 - spec/exceptional_fork_spec.rb
 - spec/spec_helper.rb