Skip to content

Exposed git_merge_base as CommitCollection.GetCommonAncestor(...) #151

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

Closed
wants to merge 3 commits into from
Closed

Exposed git_merge_base as CommitCollection.GetCommonAncestor(...) #151

wants to merge 3 commits into from

Conversation

redoz
Copy link
Contributor

@redoz redoz commented May 15, 2012

In order to make this work, I had to update libgit2 (to 54695f4098c75801f477f5bc229a52653484e08a)
When I did that I had to rename git_lasterror to giterr_last. I also had to edit some of the error codes to make sure they matched the codes given in errors.h.

Updated libgit2 to 54695f4098c75801f477f5bc229a52653484e08a
Updated libgit2 error codes to prevent test failures.
Renamed git_lasterror to giterr_last (per libgit2)
@redoz
Copy link
Contributor Author

redoz commented May 15, 2012

This currently fails some tests, I suspect that's due to the libgit2 update, investigating.
Looks like the error handling changed in libgit2, I've tried to introduce a GitError struct to do the marshalling, but I'm unable to get that working, the "message" coming back is just garbled data.

@redoz
Copy link
Contributor Author

redoz commented May 15, 2012

Unable to get the GitError working, there's also a few other failing tests due to the libgit2 update.

@redoz redoz closed this May 15, 2012
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

Successfully merging this pull request may close these issues.

1 participant