Ignore stdout during GitDownloadStrategy.stage

Library/Homebrew/download_strategy.rb

@@ -140,7 +140,7 @@ class GitDownloadStrategy <AbstractDownloadStrategy
       safe_system 'git', 'clone', @url, @clone
     else
       # TODO git pull?
-      puts "Repository already cloned"
+      puts "Repository already cloned to #{@clone}"
     end
   end
   def stage
@@ -150,9 +150,9 @@ class GitDownloadStrategy <AbstractDownloadStrategy
         ohai "Checking out #{@spec} #{@ref}"
         case @spec
         when :branch
-          safe_system 'git', 'checkout', "origin/#{@ref}"
+          nostdout { safe_system 'git', 'checkout', "origin/#{@ref}" }
         when :tag
-          safe_system 'git', 'checkout', @ref
+          nostdout { safe_system 'git', 'checkout', @ref }
         end
       end
       # http://stackoverflow.com/questions/160608/how-to-do-a-git-export-like-svn-export

Library/Homebrew/test/unittest.rb

@@ -136,21 +136,6 @@ class RefreshBrewMock < RefreshBrew
   end
 end
 
-def nostdout
-  if ARGV.include? '-V'
-    yield
-  end
-  begin
-    require 'stringio'
-    tmpo=$stdout
-    tmpe=$stderr
-    $stdout=StringIO.new
-    yield
-  ensure
-    $stdout=tmpo
-  end
-end
-
 module ExtendArgvPlusYeast
   def reset
     @named = nil

Library/Homebrew/utils.rb

@@ -159,3 +159,18 @@ def ignore_interrupts
 ensure
   trap("INT", std_trap)
 end
+
+def nostdout
+  if ARGV.verbose?
+    yield
+  else
+    begin
+      require 'stringio'
+      real_stdout = $stdout
+      $stdout = StringIO.new
+      yield
+    ensure
+      $stdout = real_stdout
+    end
+  end
+end