Catching exit status in the right place so that it works.

joakimk · Apr 27, 2012 · 43f17a9 · 43f17a9
1 parent eca76b0
commit 43f17a9
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/lib/runner/job.rb b/lib/runner/job.rb
@@ -9,6 +9,7 @@ class Job
     def initialize(runner, id, build_id, project, root, type, ruby_interpreter, files)
       @runner, @id, @build_id, @project, @root, @type, @ruby_interpreter, @files =
         runner, id, build_id, project, root, type, ruby_interpreter, files
+      @success = true
     end
 
     def jruby?
@@ -84,15 +85,16 @@ def spawn_process(command)
       @pid = POSIX::Spawn::spawn(command, :err => write_pipe, :out => write_pipe, :pgroup => true)
 
       Thread.new do
-        Process.wait(@pid)
+        Process.waitpid(@pid)
+        @success = ($?.exitstatus == 0)
         write_pipe.close
       end
 
       read_pipe
     end
 
     def success?
-      $?.exitstatus == 0
+      @success
     end
 
     def ruby_cmd