Merge pull request #1700 from MikeMcQuaid/no-libressl-and-openssl

audit: don't allow openssl & libressl dependency.
This commit is contained in:
Mike McQuaid 2016-12-29 12:32:39 +00:00 committed by GitHub
commit 27695ffeeb

View File

@ -788,6 +788,10 @@ class FormulaAuditor
problem "Please set plist_options when using a formula-defined plist."
end
if text =~ /depends_on\s+['"]openssl['"]/ && text =~ /depends_on\s+['"]libressl['"]/
problem "Formulae should not depend on both OpenSSL and LibreSSL (even optionally)."
end
return unless text.include?('require "language/go"') && !text.include?("go_resource")
problem "require \"language/go\" is unnecessary unless using `go_resource`s"
end