diff options
| author | Max Howell | 2009-10-24 15:57:23 +0100 |
|---|---|---|
| committer | Max Howell | 2009-10-24 16:20:58 +0100 |
| commit | ffe4f25d87741199cfba2052ff810712ef94854b (patch) | |
| tree | fa5613e285c93d325c105286bf8c839f0d1cc7e3 /Library | |
| parent | fc9ea77bb3c5e7c3ce2852be181fadf1b49cd304 (diff) | |
| download | brew-ffe4f25d87741199cfba2052ff810712ef94854b.tar.bz2 | |
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.
Diffstat (limited to 'Library')
| -rwxr-xr-x | Library/Homebrew/install.rb | 13 |
1 files changed, 10 insertions, 3 deletions
diff --git a/Library/Homebrew/install.rb b/Library/Homebrew/install.rb index f3ddd34bb..927e976bc 100755 --- a/Library/Homebrew/install.rb +++ b/Library/Homebrew/install.rb @@ -137,9 +137,16 @@ def install f puts rescue Exception => e - #TODO propogate exception back to brew script - onoe e - puts e.backtrace + if ENV['HOMEBREW_ERROR_PIPE'] + pipe = IO.new(ENV['HOMEBREW_ERROR_PIPE'].to_i, 'w') + Marshal.dump(e, pipe) + pipe.close + exit! 1 + else + onoe e + puts e.backtrace + exit! 2 + end end |
