@@ -1,5 +1,6 @@
 # Author:: Seth Falcon (<seth@opscode.com>)
-# Copyright:: Copyright 2010 Opscode, Inc.
+# Author:: Christopher Walters (<cw@opscode.com>)
+# Copyright:: Copyright 2010-2011 Opscode, Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -39,6 +40,17 @@ def <=>(v)
       0
     end
 
+    def hash
+      # Didn't put any thought or research into this, probably can be
+      # done better
+      to_s.hash
+    end
+
+    # For hash
+    def eql?(other)
+      other.is_a?(Version) && self == other
+    end
+    
     private
 
     def parse(str="")
 
@@ -1,5 +1,6 @@
 # Author:: Seth Falcon (<seth@opscode.com>)
-# Copyright:: Copyright 2010 Opscode, Inc.
+# Author:: Christopher Walters (<cw@opscode.com>)
+# Copyright:: Copyright 2010-2011 Opscode, Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -22,6 +23,8 @@ class VersionConstraint
     OPS = %w(< > = <= >= ~>)
     PATTERN = /^(#{OPS.join('|')}) (.+)$/
 
+    attr_reader :op, :version
+
     def initialize(constraint_spec=DEFAULT_CONSTRAINT)
       case constraint_spec
       when nil
@@ -45,6 +48,21 @@ def include?(v)
      do_op(version)
     end
 
+    def inspect
+      "(#{@op} #{@version})"
+    end
+
+    def to_s
+      "#{@op} #{@version}"
+    end
+
+    def eql?(o)
+      o.class == self.class && @op == o.op && @version == o.version
+    end
+    alias_method :==, :eql?
+
+    private
+
     def do_op(other_version)
       if STANDARD_OPS.include? @op
         other_version.send(@op.to_sym, @version)
@@ -64,17 +82,6 @@ def do_op(other_version)
       end
     end
 
-    def inspect
-      "(#{@op} #{@version})"
-    end
-
-    def to_s
-      "#{@op} #{@version}"
-    end
-
-
-    private
-
     def parse_from_array(constraint_spec)
       if constraint_spec.empty?
         parse(DEFAULT_CONSTRAINT)