(#7797) Makes git tags actually work.
authorCody Herriges <c.a.herriges@gmail.com>
Tue, 24 May 2011 09:24:26 +0000 (21:24 +1200)
committerCody Herriges <c.a.herriges@gmail.com>
Mon, 6 Jun 2011 17:42:24 +0000 (10:42 -0700)
  Fixes a number of issues with the git provider.  remote_branch_revision?
  method was always returning true because it would always at least return
  something, even if that something was a zero length array.  You normally
  don't desire a tag to become a branch since it creates ambiguity so I
  removed that.  latest method had no concept of (no branch) and so would
  fail if you switched to a remote branch or a tag.  Then finally
  revision sha1 returned by 'git rev-parse' for tags is not the revision
  of the commit the tag represents.  We have to use 'git show' and do
  some text parsing to actually figure out which commit goes with which
  tag.

lib/puppet/provider/vcsrepo/git.rb

index fa7e492..8de1313 100644 (file)
@@ -41,15 +41,22 @@ Puppet::Type.type(:vcsrepo).provide(:git, :parent => Puppet::Provider::Vcsrepo)
     branch = on_branch?
     if branch == 'master'
       return get_revision('origin/HEAD')
+    elsif branch == '(no branch)'
+      return get_revision('HEAD')
     else
-        return get_revision('origin/%s' % branch)
+      return get_revision('origin/%s' % branch)
     end
   end
 
   def revision
     update_references
-    current   = at_path { git('rev-parse', 'HEAD') }
-    canonical = at_path { git('rev-parse', @resource.value(:revision)) }
+    current = at_path { git('rev-parse', 'HEAD').chomp }
+    if tag_revision?(@resource.value(:revision))
+      canonical = at_path { git('show', @resource.value(:revision)).scan(/commit (.*)/).to_s }
+    else
+      canonical = at_path { git('rev-parse', @resource.value(:revision)).chomp }
+    end
+
     if current == canonical
       @resource.value(:revision)
     else
@@ -179,10 +186,10 @@ Puppet::Type.type(:vcsrepo).provide(:git, :parent => Puppet::Provider::Vcsrepo)
   end
 
   def checkout_or_reset(revision = @resource.value(:revision))
-    if local_branch_revision? 
+    if local_branch_revision?
       reset(revision)
     elsif tag_revision?
-      at_path { git('checkout', '-b', revision) }
+      at_path { git('checkout', revision) }
     elsif remote_branch_revision?
       at_path { git('checkout', '-b', revision, '--track', "origin/#{revision}") }
     end
@@ -206,7 +213,10 @@ Puppet::Type.type(:vcsrepo).provide(:git, :parent => Puppet::Provider::Vcsrepo)
   def remote_branch_revision?(revision = @resource.value(:revision))
     # git < 1.6 returns 'origin/#{revision}'
     # git 1.6+ returns 'remotes/origin/#{revision}'
-    at_path { branches.grep /(remotes\/)?origin\/#{revision}/ }
+    branch = at_path { branches.grep /(remotes\/)?origin\/#{revision}/ }
+    if branch.length > 0
+      return branch
+    end
   end
 
   def local_branch_revision?(revision = @resource.value(:revision))