From ffe4f25d87741199cfba2052ff810712ef94854b Mon Sep 17 00:00:00 2001
From: Max Howell
Date: Sat, 24 Oct 2009 15:57:23 +0100
Subject: Marshal install.rb exceptions back to the parent process

Using an error pipe.

The use of ENV[HOMEBREW_ERROR_PIPE] feels wrong, but I wasn't sure how else to
proxy the file descriptor to the child process since the fork immediately
calls exec.
---
 bin/brew | 22 +++++++++++++---------
 1 file changed, 13 insertions(+), 9 deletions(-)

(limited to 'bin')

diff --git a/bin/brew b/bin/brew
index 3e756ccb9..2195e3a20 100755
--- a/bin/brew
+++ b/bin/brew
@@ -163,17 +163,21 @@ begin
           #    the easiest way to do this
           # 2. formulae have access to __END__ the only way to allow this is
           #    to make the formula script the executed script
-          pid=fork
-          if pid.nil?
-            exec '/usr/bin/ruby', '-I', homebrew_rubylib_path, '-rinstall', f.path, '--', *ARGV.options
-          else
-            Process.wait pid
-          end
-          #FIXME I don't think $? represents the exit code from the child fork…
-          exit! $? if $? != 0 # exception in other brew will be visible on screen
+        read, write = IO.pipe
+        # I'm guessing this is not a good way to do this, but I'm no UNIX guru
+        ENV['HOMEBREW_ERROR_PIPE'] = write.to_i.to_s
+
+        if not fork
+          read.close
+          exec '/usr/bin/ruby', '-I', homebrew_rubylib_path, '-rinstall', f.path, '--', *ARGV.options
+        else
+          write.close
+          data = read.read
+          raise Marshal.load(data) unless data.nil? or data.empty?
+          Process.wait
         end
       end
-
+    end
 
     when 'up', 'update'
       require 'update'
-- 
cgit v1.2.3