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

check_non_libraries: *.so.* is valid on Linux

Browse Source
This commit is contained in:
Shaun Jackman 2018-08-14 11:55:39 -07:00
parent 2c68176ac2
commit 27fb8befd0
3 changed files with 18 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
Library/Homebrew/extend/os/formula_cellar_checks.rb
View File

@ -1 +1,5 @@
require "extend/os/mac/formula_cellar_checks" if OS.mac? if OS.mac?
require "extend/os/mac/formula_cellar_checks"
elsif OS.linux?
require "extend/os/linux/formula_cellar_checks"
end

5
Library/Homebrew/extend/os/linux/formula_cellar_checks.rb Normal file
View File

@ -0,0 +1,5 @@
module FormulaCellarChecks
def valid_library_extension?(filename)
generic_valid_library_extension?(filename) || filename.basename.to_s.include?(".so.")
end
end

11
Library/Homebrew/formula_cellar_checks.rb
View File

@ -56,14 +56,19 @@ module FormulaCellarChecks
EOS EOS
end end
VALID_LIBRARY_EXTENSIONS = %w[.a .dylib .framework .jnilib .la .o .so .jar .prl .pm .sh].freeze
def valid_library_extension?(filename)
VALID_LIBRARY_EXTENSIONS.include? filename.extname
end
alias generic_valid_library_extension? valid_library_extension?
def check_non_libraries def check_non_libraries
return unless formula.lib.directory? return unless formula.lib.directory?
valid_extensions = %w[.a .dylib .framework .jnilib .la .o .so
.jar .prl .pm .sh]
non_libraries = formula.lib.children.reject do |g| non_libraries = formula.lib.children.reject do |g|
next true if g.directory? next true if g.directory?
valid_extensions.include? g.extname valid_library_extension? g
end end
return if non_libraries.empty? return if non_libraries.empty?