diff options
| author | Jack Nagel | 2014-06-27 21:58:15 -0500 |
|---|---|---|
| committer | Jack Nagel | 2014-06-27 21:58:15 -0500 |
| commit | b97ff3fce4a9904ab713a61a8dccd6e2a3ad7d82 (patch) | |
| tree | c885bf5b71a7fcec91f43734fa5bd061fea77044 /Library/Homebrew/cmd | |
| parent | fb30a4ab1ffbee9268806f8935b2630ebb4fec33 (diff) | |
| download | homebrew-b97ff3fce4a9904ab713a61a8dccd6e2a3ad7d82.tar.bz2 | |
Simplify valid_formula_location? method
Diffstat (limited to 'Library/Homebrew/cmd')
| -rw-r--r-- | Library/Homebrew/cmd/update.rb | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/Library/Homebrew/cmd/update.rb b/Library/Homebrew/cmd/update.rb index 6166571cd..042214ee3 100644 --- a/Library/Homebrew/cmd/update.rb +++ b/Library/Homebrew/cmd/update.rb @@ -232,13 +232,14 @@ class Report < Hash end def valid_formula_location?(relative_path) - ruby_file = /\A.*\.rb\Z/ parts = relative_path.split('/')[2..-1] - [ - parts.length == 1 && parts.first =~ ruby_file, - parts.length == 2 && parts.first == 'Formula' && parts.last =~ ruby_file, - parts.length == 2 && parts.first == 'HomebrewFormula' && parts.last =~ ruby_file, - ].any? + return false unless File.extname(parts.last) == ".rb" + case parts.first + when "Formula", "HomebrewFormula" + parts.length == 2 + else + parts.length == 1 + end end def new_tapped_formula |
