diff options
| author | Jack Nagel | 2013-04-13 17:40:14 -0500 |
|---|---|---|
| committer | Jack Nagel | 2013-04-13 17:40:14 -0500 |
| commit | a05bb488073cc8974d263576ca61706c76e110d0 (patch) | |
| tree | 1a477ae3c9d49d161505e3842184d877eeb436f6 /Library | |
| parent | 25c0ecfd63270939aef8102e11afea161915f8c2 (diff) | |
| download | brew-a05bb488073cc8974d263576ca61706c76e110d0.tar.bz2 | |
Raise a useful exception for incomplete formulae
Diffstat (limited to 'Library')
| -rw-r--r-- | Library/Homebrew/formula.rb | 4 | ||||
| -rw-r--r-- | Library/Homebrew/test/test_formula_validation.rb | 5 |
2 files changed, 8 insertions, 1 deletions
diff --git a/Library/Homebrew/formula.rb b/Library/Homebrew/formula.rb index cf7b50e55..420203167 100644 --- a/Library/Homebrew/formula.rb +++ b/Library/Homebrew/formula.rb @@ -73,9 +73,11 @@ class Formula when @head && ARGV.build_head? then @head # --HEAD when @devel && ARGV.build_devel? then @devel # --devel when @bottle && install_bottle?(self) then @bottle # bottle available + when @stable then @stable when @devel && @stable.nil? then @devel # devel-only when @head && @stable.nil? then @head # head-only - else @stable + else + raise "Formulae require at least a URL" end end diff --git a/Library/Homebrew/test/test_formula_validation.rb b/Library/Homebrew/test/test_formula_validation.rb index 5c6bf2888..22b0eb06c 100644 --- a/Library/Homebrew/test/test_formula_validation.rb +++ b/Library/Homebrew/test/test_formula_validation.rb @@ -77,4 +77,9 @@ class FormulaValidationTests < Test::Unit::TestCase end end end + + def test_empty_formula_invalid + e = assert_raises(RuntimeError) { formula {} } + assert_equal "Formulae require at least a URL", e.message + end end |
