diff options
| author | Adam Vandenberg | 2010-08-21 11:55:57 -0700 |
|---|---|---|
| committer | Adam Vandenberg | 2010-08-21 12:00:38 -0700 |
| commit | c905fb79f41018f95889b7e8c41d107f550406c1 (patch) | |
| tree | 1dc419f9ef0a36e4f830854e7a01bf394bca7371 /Library | |
| parent | c9d41345d0413f98c2a560f9ead4d79025c9783c (diff) | |
| download | brew-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-x | Library/Contributions/examples/brew-audit.rb | 23 |
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 } |
