diff options
| author | Adam Vandenberg | 2013-03-15 15:18:22 -0700 |
|---|---|---|
| committer | Adam Vandenberg | 2013-03-15 15:41:22 -0700 |
| commit | effea228a337d92f6515231bdfc07e8a17a2e255 (patch) | |
| tree | 4aef88ce17c2b4e9191b1b096b539424d373d50e /Library/Homebrew/cmd | |
| parent | ad6abf776e0465606f3ebc04d3b68ce585217a67 (diff) | |
| download | homebrew-effea228a337d92f6515231bdfc07e8a17a2e255.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 |
