Merge pull request #3650 from MikeMcQuaid/audit-allow-std

audit: allow env :std in custom taps.
This commit is contained in:
Mike McQuaid 2018-01-09 18:12:15 +00:00 committed by GitHub
commit 311aa724dd
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -304,7 +304,7 @@ class FormulaAuditor
def audit_formula_name
return unless @strict
# skip for non-official taps
return if formula.tap.nil? || !formula.tap.official?
return unless formula.tap&.official?
name = formula.name
@ -718,7 +718,13 @@ class FormulaAuditor
return unless @strict
problem "`#{Regexp.last_match(1)}` in formulae is deprecated" if line =~ /(env :(std|userpaths))/
if formula.tap&.official? && line.include?("env :std")
problem "`env :std` in official tap formulae is deprecated"
end
if line.include?("env :userpaths")
problem "`env :userpaths` in formulae is deprecated"
end
if line =~ /system ((["'])[^"' ]*(?:\s[^"' ]*)+\2)/
bad_system = Regexp.last_match(1)