When a full commit SHA is provided as revision, provide it with the git clone command and honor the depth

Signed-off-by: Sascha Schwarze <[email protected]>

Author: SaschaSchwarze0

cmd/git/main.go

@@ -69,8 +69,9 @@ type settings struct {
 var flagValues settings
 
 var (
-	sshGitURLRegEx = regexp.MustCompile(`^(git@|ssh:\/\/).+$`)
-	commitShaRegEx = regexp.MustCompile(`^[0-9a-f]{7,40}$`)
+	sshGitURLRegEx      = regexp.MustCompile(`^(git@|ssh:\/\/).+$`)
+	fullCommitShaRegex  = regexp.MustCompile(`^[0-9a-f]{40}$`)
+	shortCommitShaRegEx = regexp.MustCompile(`^[0-9a-f]{7,39}$`)
 )
 
 func init() {
@@ -273,10 +274,19 @@ func clone(ctx context.Context) error {
 		cloneArgs = append(cloneArgs, "--no-tags")
 	}
 
-	var commitSha string
+	var shortCommitSha string
 	switch {
-	case commitShaRegEx.MatchString(flagValues.revision):
-		commitSha = flagValues.revision
+	case fullCommitShaRegex.MatchString(flagValues.revision):
+		// we can pass the commit SHA directly to `git clone --revision` and can honor the depth.
+		cloneArgs = append(cloneArgs, "--revision", flagValues.revision)
+
+		if flagValues.depth > 0 {
+			cloneArgs = append(cloneArgs, "--depth", fmt.Sprintf("%d", flagValues.depth))
+		}
+
+	case shortCommitShaRegEx.MatchString(flagValues.revision):
+		// for a short commit SHA we must do a full `git clone --no-checkout` and later a checkout. For this, we cannot honor depth.
+		shortCommitSha = flagValues.revision
 		cloneArgs = append(cloneArgs, "--no-checkout")
 
 	default:
@@ -417,8 +427,8 @@ func clone(ctx context.Context) error {
 		return err
 	}
 
-	if commitSha != "" {
-		if _, err := git(ctx, "-C", flagValues.target, "checkout", commitSha); err != nil {
+	if shortCommitSha != "" {
+		if _, err := git(ctx, "-C", flagValues.target, "checkout", shortCommitSha); err != nil {
 			return err
 		}
 	}