admin/brew
1
0
Fork 0
Code Issues Packages Projects Releases 6 Wiki Activity

Complain about non-exes in bin and non-libs in lib

Browse Source
This commit is contained in:
Adam Vandenberg 2012-02-19 19:09:38 -08:00
parent 86e7c8a772
commit 47a81b0b80
1 changed files with 49 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

59
Library/Homebrew/formula_installer.rb
View File

@ -82,9 +82,10 @@ class FormulaInstaller
ohai 'Caveats', f.keg_only_text
@show_summary_heading = true
else
audit_bin
audit_lib
check_manpages
check_infopages
check_jars
check_m4
end
end
@ -244,18 +245,56 @@ class FormulaInstaller
def check_jars
# Check for Jars in lib
if File.exist?(f.lib)
unless f.lib.children.select{|g| g.to_s =~ /\.jar$/}.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."
@show_summary_heading = true
end
return unless File.exist? f.lib
unless f.lib.children.select{|g| g.to_s =~ /\.jar$/}.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."
@show_summary_heading = true
end
end
def check_non_libraries
return unless File.exist? f.lib
valid_libraries = %w(.a .dylib .framework .la .so)
non_libraries = f.lib.children.select do |g|
next if g.directory?
extname = g.extname
(extname != ".jar") and (not valid_libraries.include? extname)
end
unless non_libraries.empty?
opoo 'Non-libraries were installed to "lib".'
puts "Installing non-libraries to \"lib\" is bad practice."
puts "The offending files are:"
puts non_libraries
@show_summary_heading = true
end
end
def audit_bin
return unless File.exist? f.bin
non_exes = f.bin.children.select {|g| not File.executable? g}
unless non_exes.empty?
opoo 'Non-executables were installed to "bin".'
puts "Installing non-executables to \"bin\" is bad practice."
puts "The offending files are:"
puts non_exes
@show_summary_heading = true
end
end
def audit_lib
check_jars
check_non_libraries
end
def check_m4
# Check for m4 files
if Dir[f.share+"aclocal/*.m4"].length > 0 and not in_aclocal_dirlist?