Remove support for version "schemes", just pass version objects directly

I'm not sure why I thought reinventing object instantiation was a good
idea.

3 files changed, 6 insertions, 14 deletions

diff --git a/Library/Homebrew/resource.rb b/Library/Homebrew/resource.rb
index 6a72a6e99..3409845a0 100644
--- a/Library/Homebrew/resource.rb
+++ b/Library/Homebrew/resource.rb
@@ -118,9 +118,9 @@ class Resource
 
   def detect_version(val)
     case val
-    when nil    then Version.detect(url, specs)
-    when String then Version.new(val)
-    when Hash   then Version.new_with_scheme(*val.shift)
+    when nil     then Version.detect(url, specs)
+    when String  then Version.new(val)
+    when Version then val
     else
       raise TypeError, "version '#{val.inspect}' should be a string"
     end
diff --git a/Library/Homebrew/test/test_resource.rb b/Library/Homebrew/test/test_resource.rb
index 3cfe592e1..2baf4750e 100644
--- a/Library/Homebrew/test/test_resource.rb
+++ b/Library/Homebrew/test/test_resource.rb
@@ -61,10 +61,10 @@ class ResourceTests < Test::Unit::TestCase
   end
 
   def test_version_with_scheme
-    scheme = Class.new(Version)
-    @resource.version('1.0' => scheme)
+    klass = Class.new(Version)
+    @resource.version klass.new("1.0")
     assert_version_equal '1.0', @resource.version
-    assert_instance_of scheme, @resource.version
+    assert_instance_of klass, @resource.version
   end
 
   def test_version_from_tag
diff --git a/Library/Homebrew/version.rb b/Library/Homebrew/version.rb
index 2e0bc85a5..cd1c14989 100644
--- a/Library/Homebrew/version.rb
+++ b/Library/Homebrew/version.rb
@@ -155,14 +155,6 @@ class Version
     StringToken::PATTERN
   )
 
-  def self.new_with_scheme(value, scheme)
-    if Class === scheme && scheme.ancestors.include?(Version)
-      scheme.new(value)
-    else
-      raise TypeError, "Unknown version scheme #{scheme.inspect}"
-    end
-  end
-
   def self.detect(url, specs={})
     if specs.has_key?(:tag)
       new(specs[:tag][/((?:\d+\.)*\d+)/, 1], true)