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.

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

Doesn't parse packed refs #27

Open
spheromak opened this issue Nov 7, 2016 · 5 comments
Open

Doesn't parse packed refs #27

spheromak opened this issue Nov 7, 2016 · 5 comments

Comments

@spheromak
Copy link

Operations only work on repos that have 'loose' refs to tags i.e. have not been GC'd. Probably also goes for packed heads/remotes

@spheromak
Copy link
Author

that might be the case of the lower level object implementation but the
high level repo.GetTags() will not find any tags if the repo has been gc'd
or if it's a fresh checkout of a packed repo.

https://github.com/gogits/git/blob/master/repo_tag.go#L20

On Mon, Nov 7, 2016 at 6:36 PM, Lunny Xiao notifications@github.com wrote:

It supports packed refs, you can find the code here
https://github.com/gogits/git/blob/master/repo_object.go#L73.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#27 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAC81BBWpLq4zoKuwCh2e7t5dXd4TvKnks5q7-AWgaJpZM4KroDw
.

@spheromak spheromak changed the title Doens't parse packed refs Doesn't parse packed refs Nov 11, 2016
@lemmi
Copy link

lemmi commented Jun 13, 2017

@spheromak #28 was just pushed and reworked the handling of packed files. Does this maybe fix this issue?

@necrophonic
Copy link

Hi, I was wondering if anyone has had time to take a look at my pull request #31 attempting to fix this issue? Thanks

@lunny
Copy link

lunny commented Aug 30, 2017

@necrophonic maybe you can also send to https://github.com/go-gitea/git

@spheromak
Copy link
Author

I think #28 fixed it. I went back to using the git bunary in the project I was using this lib with so I haven’t tested it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants