diff options
| author | Adam Vandenberg | 2013-03-15 15:18:22 -0700 |
|---|---|---|
| committer | Adam Vandenberg | 2013-03-15 15:41:22 -0700 |
| commit | ccd69e45d4aeedcc4ab6f1e4cc1c3cd43a871936 (patch) | |
| tree | 30ee070005bfbba2540c8b6c32c6b73b9b022e6a /Library/Homebrew/cmd | |
| parent | 5f6515baad38896668ce9750f4026e2d39e898fb (diff) | |
| download | brew-ccd69e45d4aeedcc4ab6f1e4cc1c3cd43a871936.tar.bz2 | |
audit github downloads
We want the new-style "/archive/" downloads.
Diffstat (limited to 'Library/Homebrew/cmd')
| -rw-r--r-- | Library/Homebrew/cmd/audit.rb | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/Library/Homebrew/cmd/audit.rb b/Library/Homebrew/cmd/audit.rb index 890a4cf10..31dac7291 100644 --- a/Library/Homebrew/cmd/audit.rb +++ b/Library/Homebrew/cmd/audit.rb @@ -231,6 +231,11 @@ class FormulaAuditor problem "Use https:// URLs for accessing GitHub repositories (url is #{u})." end + # Use new-style archive downloads + urls.select { |u| u =~ %r[https://.*/(tar|zip)ball/] and not u =~ %r[\.git$] }.each do |u| + problem "Use /archive/ URLs for GitHub tarballs (url is #{u})." + end + if urls.any? { |u| u =~ /\.xz/ } && !f.deps.any? { |d| d.name == "xz" } problem "Missing a build-time dependency on 'xz'" end |
