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

Add allowlist, make branch: mandatory

Browse Source
This commit is contained in:
Nanda H Krishna 2021-07-23 15:33:44 +05:30
parent d75320069f
commit f69f9be233
No known key found for this signature in database
GPG Key ID: 067E5FCD58ADF3AA
2 changed files with 13 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
Library/Homebrew/formula_auditor.rb
View File

@ -523,9 +523,15 @@ module Homebrew
spec_name = name.downcase.to_sym
next unless (spec = formula.send(spec_name))
except = @except.to_a
if spec_name == :head &&
tap_audit_exception(:head_non_default_branch_allowlist, formula.name, spec.specs[:branch])
except << "head_branch"
end
ra = ResourceAuditor.new(
spec, spec_name,
online: @online, strict: @strict, only: @only, except: @except
online: @online, strict: @strict, only: @only, except: except
).audit
ra.problems.each do |message|
problem "#{name}: #{message}"

13
Library/Homebrew/resource_auditor.rb
View File

@ -125,18 +125,17 @@ module Homebrew
end
def audit_head_branch
return if !@online || !@strict || spec_name != :head || !Utils::Git.remote_exists?(url)
return unless @online
return unless @strict
return unless spec_name == :head
return unless Utils::Git.remote_exists?(url)
branch = Utils.popen_read("git", "ls-remote", "--symref", url, "HEAD")
.match(%r{ref: refs/heads/(.*?)\s+HEAD})[1]
return if branch == "master" && specs[:branch].blank? || branch == specs[:branch]
return if branch == specs[:branch]
if branch == "master"
problem "Remove `branch: \"#{specs[:branch]}\"`"
else
problem "Use `branch: \"#{branch}\"` to specify the correct default branch"
end
problem "Use `branch: \"#{branch}\"` to specify the default branch"
end
def problem(text)