From 508216ec2b3e55d24d61be5ef392693406d8d816 Mon Sep 17 00:00:00 2001 From: Max Howell Date: Thu, 16 Jun 2011 17:38:39 +0100 Subject: Show summary headings when appropriate --- Library/Homebrew/install.rb | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) (limited to 'Library') diff --git a/Library/Homebrew/install.rb b/Library/Homebrew/install.rb index ca4e6663b..b07d2371b 100755 --- a/Library/Homebrew/install.rb +++ b/Library/Homebrew/install.rb @@ -152,6 +152,7 @@ def install f opoo 'A top-level "man" folder was found.' puts "Homebrew requires that man pages live under share." puts 'This can often be fixed by passing "--mandir=#{man}" to configure.' + show_summary_heading = true end # Check for info pages that aren't in share/info @@ -159,6 +160,7 @@ def install f opoo 'A top-level "info" folder was found.' puts "Homebrew suggests that info pages live under share." puts 'This can often be fixed by passing "--infodir=#{info}" to configure.' + show_summary_heading = true end # Check for Jars in lib @@ -169,6 +171,7 @@ def install f puts "For Java software, it is typically better for the formula to" puts "install to \"libexec\" and then symlink or wrap binaries into \"bin\"." puts "See \"activemq\", \"jruby\", etc. for examples." + show_summary_heading = true end end @@ -178,6 +181,7 @@ def install f puts "Homebrew does not append \"#{HOMEBREW_PREFIX}/share/aclocal\"" puts "to \"/usr/share/aclocal/dirlist\". If an autoconf script you use" puts "requires these m4 macros, you'll need to add this path manually." + show_summary_heading = true end # link from Cellar to Prefix @@ -187,11 +191,7 @@ def install f onoe "The linking step did not complete successfully" puts "The formula built, but is not symlinked into #{HOMEBREW_PREFIX}" puts "You can try again using `brew link #{f.name}'" - if ARGV.debug? - ohai e, e.backtrace - else - onoe e - end + ohai e, e.backtrace if ARGV.debug? show_summary_heading = true end -- cgit v1.2.3