diff options
| author | Misty De Meo | 2012-11-28 10:00:40 -0600 |
|---|---|---|
| committer | Misty De Meo | 2012-11-28 10:01:00 -0600 |
| commit | a171ec18fe0b198613b8cc198118d8573cba0289 (patch) | |
| tree | a6a59d6a39d2bc9c7713ae30bb4b9bbd2866fa7f /Library/Homebrew | |
| parent | ff356b6192d323afb6493c906896f69183ba3ffe (diff) | |
| download | brew-a171ec18fe0b198613b8cc198118d8573cba0289.tar.bz2 | |
Formula.factory: raise on encountering non-Formula
Formula.factory naively assumed that any already-loaded constant
fed into it was a formula, with confusing results when trying to
`brew install` a name from the ruby stdlib.
Fixes Homebrew/homebrew#16284.
Diffstat (limited to 'Library/Homebrew')
| -rw-r--r-- | Library/Homebrew/formula.rb | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/Library/Homebrew/formula.rb b/Library/Homebrew/formula.rb index 3f5f83ced..cb78221fd 100644 --- a/Library/Homebrew/formula.rb +++ b/Library/Homebrew/formula.rb @@ -395,6 +395,8 @@ class Formula raise LoadError end + raise NameError if klass.superclass != Formula + return klass.new(name) if install_type == :from_name return klass.new(name, path.to_s) rescue NoMethodError |