aboutsummaryrefslogtreecommitdiffstats
path: root/Library
diff options
context:
space:
mode:
authorAdam Vandenberg2010-08-21 11:55:57 -0700
committerAdam Vandenberg2010-08-21 12:00:38 -0700
commitc905fb79f41018f95889b7e8c41d107f550406c1 (patch)
tree1dc419f9ef0a36e4f830854e7a01bf394bca7371 /Library
parentc9d41345d0413f98c2a560f9ead4d79025c9783c (diff)
downloadbrew-c905fb79f41018f95889b7e8c41d107f550406c1.tar.bz2
brew audit - Google Code homes w/o trailing slash
Without a trailing slash, you redirect to the URL w/ a slash, so might as well have it there in the first place.
Diffstat (limited to 'Library')
-rwxr-xr-xLibrary/Contributions/examples/brew-audit.rb23
1 files changed, 18 insertions, 5 deletions
diff --git a/Library/Contributions/examples/brew-audit.rb b/Library/Contributions/examples/brew-audit.rb
index 2afcb05c6..b968ef327 100755
--- a/Library/Contributions/examples/brew-audit.rb
+++ b/Library/Contributions/examples/brew-audit.rb
@@ -103,15 +103,28 @@ def audit_formula_options f, text
return problems
end
+def audit_formula_instance f
+ problems = []
+
+ # Don't depend_on aliases; use full name
+ aliases = Formula.aliases
+ f.deps.select {|d| aliases.include? d}.each do |d|
+ problems << " * Dep #{d} is an alias; switch to the real name."
+ end
+
+ # Google Code homepages should end in a slash
+ if f.homepage =~ %r[^https?://code\.google\.com/p/[^/]+[^/]$]
+ problems << " * Google Code homepage should end with a slash."
+ end
+
+ return problems
+end
+
def audit_some_formulae
ff.each do |f|
problems = []
- # Don't depend_on aliases; use full name
- aliases = Formula.aliases
- f.deps.select {|d| aliases.include? d}.each do |d|
- problems << " * Dep #{d} is an alias; switch to the real name."
- end
+ problems += audit_formula_instance f
text = ""
File.open(f.path, "r") { |afile| text = afile.read }