diff options
| author | Misty De Meo | 2017-06-29 19:57:14 -0700 |
|---|---|---|
| committer | GitHub | 2017-06-29 19:57:14 -0700 |
| commit | 0077e4f20ef9d107a0ca8c5ab75cac552163fb45 (patch) | |
| tree | ed53b218bddd02a90b7c3e188790ace6cdbbe21f /Library | |
| parent | 1503c9fcb2509d099dea8fa0f1ae36b9044152f5 (diff) | |
| parent | 87fa20076c770e78516265507dd183cc6abc9c00 (diff) | |
| download | brew-0077e4f20ef9d107a0ca8c5ab75cac552163fb45.tar.bz2 | |
Merge pull request #2837 from mistydemeo/audit_request_full_index_patches
audit: ask for full_index patches
Diffstat (limited to 'Library')
| -rw-r--r-- | Library/Homebrew/dev-cmd/audit.rb | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/Library/Homebrew/dev-cmd/audit.rb b/Library/Homebrew/dev-cmd/audit.rb index 1c7476c4b..5b2edfcf6 100644 --- a/Library/Homebrew/dev-cmd/audit.rb +++ b/Library/Homebrew/dev-cmd/audit.rb @@ -819,6 +819,13 @@ class FormulaAuditor def patch_problems(patch) case patch.url + when %r{https?://github\.com/.+/.+/(?:commit|pull)/[a-fA-F0-9]*.(?:patch|diff)} + unless patch.url =~ /\?full_index=\w+$/ + problem <<-EOS.undent + GitHub patches should use the full_index parameter: + #{patch.url}?full_index=1 + EOS + end when /raw\.github\.com/, %r{gist\.github\.com/raw}, %r{gist\.github\.com/.+/raw}, %r{gist\.githubusercontent\.com/.+/raw} unless patch.url =~ /[a-fA-F0-9]{40}/ @@ -827,7 +834,7 @@ class FormulaAuditor when %r{https?://patch-diff\.githubusercontent\.com/raw/(.+)/(.+)/pull/(.+)\.(?:diff|patch)} problem <<-EOS.undent use GitHub pull request URLs: - https://github.com/#{Regexp.last_match(1)}/#{Regexp.last_match(2)}/pull/#{Regexp.last_match(3)}.patch + https://github.com/#{Regexp.last_match(1)}/#{Regexp.last_match(2)}/pull/#{Regexp.last_match(3)}.patch?full_index=1 Rather than patch-diff: #{patch.url} EOS |