diff options
| author | Jack Nagel | 2011-10-15 00:39:05 -0500 |
|---|---|---|
| committer | Jack Nagel | 2011-10-15 00:39:05 -0500 |
| commit | 5a1fcee33660be732c7841ffe71ad437e2eba5ce (patch) | |
| tree | e4c3e764487534e8cf73ee0b3621b3c555ac47d0 | |
| parent | 87718ca6d76253a45eb664e47825913bdd8e1015 (diff) | |
| download | homebrew-5a1fcee33660be732c7841ffe71ad437e2eba5ce.tar.bz2 | |
audit: check mirror URLs
This will actually throw some possibly unwanted warnings, e.g. whining
about using mirrors.kernel.org for Debian software; we might only want
that warning for the default URL.
Perhaps mirrors deserve their own audit_formula_mirrors, but rather than
duplicate code, let's just check them against the standard criteria for
now.
Signed-off-by: Jack Nagel <jacknagel@gmail.com>
| -rwxr-xr-x | 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 fce3d869e..1a57ab80a 100755 --- a/Library/Homebrew/cmd/audit.rb +++ b/Library/Homebrew/cmd/audit.rb @@ -178,6 +178,11 @@ def audit_formula_urls f urls = [(f.url rescue nil), (f.head rescue nil)].reject {|p| p.nil?} + f.mirrors.each do |m| + mirror = m.values_at :url + urls << (mirror.to_s rescue nil) + end + # Check SourceForge urls urls.each do |p| # Is it a filedownload (instead of svnroot) |