Match also partial checksums so we don't have to type the whole thing out

Author: ctennis

chef/lib/chef/provider/remote_file.rb

@@ -46,7 +46,9 @@ def do_remote_file(source, path)
         # The current files checksum
         current_checksum = self.checksum(path) if ::File.exists?(path)
 
-        unless (@new_resource.checksum && @new_resource.checksum == current_checksum)
+        if(@new_resource.checksum && current_checksum && current_checksum =~ /^#{@new_resource.checksum}/)
+          Chef::Log.debug("File #{@new_resource} checksum matches, not updating")
+        else
           begin
             # The remote filehandle
             raw_file = get_from_uri(source)    ||
@@ -82,8 +84,6 @@ def do_remote_file(source, path)
           # We're done with the file, so make sure to close it if it was open.
           raw_file.close 
           true
-        else
-          Chef::Log.debug("File #{@new_resource} checksum matches, not updating")
         end
 
         set_owner if @new_resource.owner

chef/spec/unit/provider/remote_file_spec.rb

@@ -73,12 +73,29 @@ def do_remote_file
         do_remote_file
       end
 
-      it "should download the file if the checksum matches" do
+      it "should not download the file if the checksum is a partial match from the beginning" do
+        @resource.checksum("0fd012fd")
+        @resource.source("http://opscode.com/seattle.txt")
+        @rest.should_not_receive(:get_rest).with("http://opscode.com/seattle.txt", true).and_return(@tempfile)
+        do_remote_file
+      end
+
+
+      it "should download the file if the checksum does not match" do
         @resource.checksum("this hash doesn't match")
         @resource.source("http://opscode.com/seattle.txt")
         @rest.should_receive(:get_rest).with("http://opscode.com/seattle.txt", true).and_return(@tempfile)
         do_remote_file
       end
+
+      it "should download the file if the checksum matches, but not from the beginning" do
+        @resource.checksum("fd012fd")
+        @resource.source("http://opscode.com/seattle.txt")
+        @rest.should_receive(:get_rest).with("http://opscode.com/seattle.txt", true).and_return(@tempfile)
+        do_remote_file
+      end
+
+
     end
 
     describe "and not given a checksum" do