aboutsummaryrefslogtreecommitdiffstats
path: root/Library
diff options
context:
space:
mode:
Diffstat (limited to 'Library')
-rwxr-xr-xLibrary/Homebrew/cmd/audit.rb16
1 files changed, 8 insertions, 8 deletions
diff --git a/Library/Homebrew/cmd/audit.rb b/Library/Homebrew/cmd/audit.rb
index 31c0fbaeb..d3ea129a4 100755
--- a/Library/Homebrew/cmd/audit.rb
+++ b/Library/Homebrew/cmd/audit.rb
@@ -198,6 +198,14 @@ def audit_formula_urls f
urls = [(f.url rescue nil), (f.head rescue nil)].reject {|p| p.nil?}
urls.uniq! # head-only formulae result in duplicate entries
+ # Check GNU urls; doesn't apply to mirrors
+ urls.each do |p|
+ if p =~ %r[^(https?|ftp)://(.+)/gnu/]
+ problems << " * \"ftpmirror.gnu.org\" is preferred for GNU software."
+ end
+ end
+
+ # the rest of the checks apply to mirrors as well
f.mirrors.each do |m|
mirror = m.values_at :url
urls << (mirror.to_s rescue nil)
@@ -236,14 +244,6 @@ def audit_formula_urls f
end
end
- # Check GNU urls
- # FIXME only check primary URLs, not mirrors
- # urls.each do |p|
- # if p =~ %r[^(https?|ftp)://(.+)/gnu/]
- # problems << " * \"ftpmirror.gnu.org\" is preferred for GNU software."
- # end
- # end
-
return problems
end