opscode/chef — pull request #1425 by rvalyi (2014-05-11 02:01:31+00:00)
Title: git resource status checking saves 1 shell_out system call

find_current_revision makes a system call (not the fastest thing on earth). However when checking a git repo status, find_current_revision is called twice:
once in load_current_resource that will store it in @current_resource.revision
and find_current_revision is called again later in action_sync

That last call is useless, we can simplify like I suggest.

NOTE: I actually discovered that while coding a similar provider for the BZR SCM where calling the bzr command is so much slower that I couldn't miss the issue.
If you are interested in my bzr provider, let me know, I can do the changes you may suggests
https://github.com/akretion/ak-bzr/blob/master/libraries/bzr.rb#L186