diff options
| author | Adam Vandenberg | 2012-02-25 20:57:00 -0800 |
|---|---|---|
| committer | Adam Vandenberg | 2012-02-25 20:57:21 -0800 |
| commit | 9ca233812ce95b62cf79f75efdcbe1b73cb7a1ea (patch) | |
| tree | 30bfd72439587b67eae460feb6f0cfc0e933e729 /Library | |
| parent | 42d44053ef5d9a3fd46e96be57b73a690a4501fa (diff) | |
| download | brew-9ca233812ce95b62cf79f75efdcbe1b73cb7a1ea.tar.bz2 | |
List offending jar files when installed to lib
Diffstat (limited to 'Library')
| -rw-r--r-- | Library/Homebrew/formula_installer.rb | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/Library/Homebrew/formula_installer.rb b/Library/Homebrew/formula_installer.rb index 979cf02e3..e61967bb5 100644 --- a/Library/Homebrew/formula_installer.rb +++ b/Library/Homebrew/formula_installer.rb @@ -250,15 +250,17 @@ class FormulaInstaller end def check_jars - # Check for Jars in lib return unless File.exist? f.lib - unless f.lib.children.select{|g| g.to_s =~ /\.jar$/}.empty? + jars = f.lib.children.select{|g| g.to_s =~ /\.jar$/} + unless jars.empty? opoo 'JARs were installed to "lib".' puts "Installing JARs to \"lib\" can cause conflicts between packages." 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." + puts "The offending files are:" + puts jars @show_summary_heading = true end end |
