Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

pumbling: packfile, resolve external reference delta #392

Merged
merged 8 commits into from Jan 8, 2022
Merged

pumbling: packfile, resolve external reference delta #392

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
aba274c
resolve external reference deltas
ga-paul-t Oct 11, 2021
5151448
resolve external reference delta
ga-paul-t Oct 11, 2021
411809d
Merge branch 'go-git:master' into codecommit-ref-delta
ga-paul-t Nov 2, 2021
7391aa4
add codecommit packfile for testing external ref resolution
ga-paul-t Nov 9, 2021
b0f5eb8
include example codecommit pack file
ga-paul-t Nov 9, 2021
53a714b
Merge branch 'go-git:master' into master
ga-paul-t Dec 15, 2021
8d923a6
Merge branch 'master' into codecommit-ref-delta
ga-paul-t Dec 15, 2021
bf57a4c
remove packfile and align to test fixtures
ga-paul-t Jan 5, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
11 changes: 11 additions & 0 deletions plumbing/format/packfile/parser.go
View file
Open in desktop
Expand Up @@ -287,6 +287,7 @@ func (p *Parser) resolveDeltas() error {
if err := p.resolveObject(stdioutil.Discard, child, content); err != nil {
return err
}
p.resolveExternalRef(child)
}

// Remove the delta from the cache.
Expand All @@ -299,6 +300,16 @@ func (p *Parser) resolveDeltas() error {
return nil
}

func (p *Parser) resolveExternalRef(o *objectInfo) {
if ref, ok := p.oiByHash[o.SHA1]; ok && ref.ExternalRef {
p.oiByHash[o.SHA1] = o
o.Children = ref.Children
for _, c := range o.Children {
c.Parent = o
}
}
}

func (p *Parser) get(o *objectInfo, buf *bytes.Buffer) (err error) {
if !o.ExternalRef { // skip cache check for placeholder parents
b, ok := p.cache.Get(o.Offset)
Expand Down