Support releases from annotated tags

Annotated tags have their own hash, different from the commit hash, so the tag must be resolved before checkout.

Author: per1234

libraries/git_integration_test.go

@@ -8,6 +8,7 @@ import (
 
 	"arduino.cc/repository/libraries/db"
 	"github.com/go-git/go-git/v5"
+	"github.com/go-git/go-git/v5/plumbing"
 	"github.com/stretchr/testify/require"
 )
 
@@ -41,7 +42,10 @@ func TestUpdateLibraryJson(t *testing.T) {
 
 		repoTree, err := r.Repository.Worktree()
 		require.NoError(t, err)
-		err = repoTree.Checkout(&git.CheckoutOptions{Hash: tag.Hash()})
+		// Annotated tags have their own hash, different from the commit hash, so the tag must be resolved before checkout
+		resolvedTag, err := r.Repository.ResolveRevision(plumbing.Revision(tag.Hash().String()))
+		require.NoError(t, err)
+		err = repoTree.Checkout(&git.CheckoutOptions{Hash: *resolvedTag})
 		require.NoError(t, err)
 
 		library, err := GenerateLibraryFromRepo(r)

sync_libraries.go

@@ -234,7 +234,13 @@ func syncLibraryTaggedRelease(logger *log.Logger, repo *libraries.Repository, ta
 		return err
 	}
 
-	if err = repoTree.Checkout(&git.CheckoutOptions{Hash: tag.Hash()}); err != nil {
+	// Annotated tags have their own hash, different from the commit hash, so the tag must be resolved before checkout
+	resolvedTag, err := repo.Repository.ResolveRevision(plumbing.Revision(tag.Hash().String()))
+	if err != nil {
+		panic(err)
+	}
+
+	if err = repoTree.Checkout(&git.CheckoutOptions{Hash: *resolvedTag}); err != nil {
 		return fmt.Errorf("Error checking out repo: %s", err)
 	}