From a3409a179e3c56e6e8636441bc61e2efd765332d Mon Sep 17 00:00:00 2001
From: Jack Nagel <jacknagel@gmail.com>
Date: Fri, 5 Sep 2014 15:13:53 -0500
Subject: [PATCH] Reduce branches and effective scope of some variables

---
 Library/Homebrew/formula.rb | 50 +++++++++++++++++--------------------
 1 file changed, 23 insertions(+), 27 deletions(-)

diff --git a/Library/Homebrew/formula.rb b/Library/Homebrew/formula.rb
index 7d8f996023..b65d56dd70 100644
--- a/Library/Homebrew/formula.rb
+++ b/Library/Homebrew/formula.rb
@@ -513,40 +513,36 @@ class Formula
 
     if ARGV.verbose?
       rd, wr = IO.pipe
-      out = wr
-    else
-      out = log
-    end
-
-    begin
-      pid = fork do
-        if rd
+      begin
+        pid = fork do
           rd.close
           log.close
+          exec_cmd(cmd, args, wr, logfn)
         end
-        exec_cmd(cmd, args, out, logfn)
+        wr.close
+
+        while buf = rd.gets
+          log.puts buf
+          puts buf
+        end
+      ensure
+        rd.close unless rd.closed?
       end
-      wr.close if wr
+    else
+      pid = fork { exec_cmd(cmd, args, log, logfn) }
+    end
 
-      while buf = rd.gets
-        log.puts buf
-        puts buf
-      end if rd
+    Process.wait(pid)
 
-      Process.wait(pid)
+    $stdout.flush
 
-      $stdout.flush
-
-      unless $?.success?
-        log.flush
-        Kernel.system "/usr/bin/tail", "-n", "5", logfn unless ARGV.verbose?
-        log.puts
-        require 'cmd/config'
-        Homebrew.dump_build_config(log)
-        raise BuildError.new(self, cmd, args)
-      end
-    ensure
-      rd.close if rd && !rd.closed?
+    unless $?.success?
+      log.flush
+      Kernel.system "/usr/bin/tail", "-n", "5", logfn unless ARGV.verbose?
+      log.puts
+      require 'cmd/config'
+      Homebrew.dump_build_config(log)
+      raise BuildError.new(self, cmd, args)
     end
   end