diff options
author | Paulo Gomes <pjbgf@linux.com> | 2023-06-07 23:56:59 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-06-07 23:56:59 +0100 |
commit | 35f7e6770361a2c16c9b6c44acdc38ae04c75bd3 (patch) | |
tree | c26169956c284d3fb7fc21ec0a907997dd3b2739 | |
parent | 42112780406835a867e6e8cce5ccca3adf13c803 (diff) | |
parent | 2d6af16bf6d051cb3014c9970f3ea813e54f73b0 (diff) | |
download | go-git-35f7e6770361a2c16c9b6c44acdc38ae04c75bd3.tar.gz |
Merge pull request #765 from matejrisek/feature/shallow-submodules-option
git: add a clone option to allow for shallow cloning of submodules
-rw-r--r-- | options.go | 3 | ||||
-rw-r--r-- | repository.go | 8 | ||||
-rw-r--r-- | repository_test.go | 37 |
3 files changed, 47 insertions, 1 deletions
@@ -62,6 +62,9 @@ type CloneOptions struct { // within, using their default settings. This option is ignored if the // cloned repository does not have a worktree. RecurseSubmodules SubmoduleRescursivity + // ShallowSubmodules limit cloning submodules to the 1 level of depth. + // It matches the git command --shallow-submodules. + ShallowSubmodules bool // Progress is where the human readable information sent by the server is // stored, if nil nothing is stored and the capability (if supported) // no-progress, is sent to the server to avoid send this information. diff --git a/repository.go b/repository.go index 6c3be2e..dd822a5 100644 --- a/repository.go +++ b/repository.go @@ -916,7 +916,13 @@ func (r *Repository) clone(ctx context.Context, o *CloneOptions) error { if o.RecurseSubmodules != NoRecurseSubmodules { if err := w.updateSubmodules(&SubmoduleUpdateOptions{ RecurseSubmodules: o.RecurseSubmodules, - Auth: o.Auth, + Depth: func() int { + if o.ShallowSubmodules { + return 1 + } + return 0 + }(), + Auth: o.Auth, }); err != nil { return err } diff --git a/repository_test.go b/repository_test.go index bcfaa93..d18816f 100644 --- a/repository_test.go +++ b/repository_test.go @@ -932,6 +932,43 @@ func (s *RepositorySuite) TestPlainCloneWithRecurseSubmodules(c *C) { c.Assert(cfg.Submodules, HasLen, 2) } +func (s *RepositorySuite) TestPlainCloneWithShallowSubmodules(c *C) { + if testing.Short() { + c.Skip("skipping test in short mode.") + } + + dir, clean := s.TemporalDir() + defer clean() + + path := fixtures.ByTag("submodule").One().Worktree().Root() + mainRepo, err := PlainClone(dir, false, &CloneOptions{ + URL: path, + RecurseSubmodules: 1, + ShallowSubmodules: true, + }) + c.Assert(err, IsNil) + + mainWorktree, err := mainRepo.Worktree() + c.Assert(err, IsNil) + + submodule, err := mainWorktree.Submodule("basic") + c.Assert(err, IsNil) + + subRepo, err := submodule.Repository() + c.Assert(err, IsNil) + + lr, err := subRepo.Log(&LogOptions{}) + c.Assert(err, IsNil) + + commitCount := 0 + for _, err := lr.Next(); err == nil; _, err = lr.Next() { + commitCount++ + } + c.Assert(err, IsNil) + + c.Assert(commitCount, Equals, 1) +} + func (s *RepositorySuite) TestPlainCloneNoCheckout(c *C) { dir, clean := s.TemporalDir() defer clean() |