Skip to content

Move blame to cffi#413

Merged
jdavid merged 1 commit intolibgit2:masterfrom
carlosmn:cmn/cffi-blame
Sep 1, 2014
Merged

Move blame to cffi#413
jdavid merged 1 commit intolibgit2:masterfrom
carlosmn:cmn/cffi-blame

Conversation

@carlosmn
Copy link
Member

This requires fairly little work on the pygit2 side to kick off all the
searching on the libgit2 side, so it's a fairly good candidate.

This changes the return value for the commit ids to Oid instead of
strings, which is what we generally try to return.

@jdavid
Copy link
Member

jdavid commented Aug 28, 2014

Travis reports build error.

@carlosmn
Copy link
Member Author

Looks like I didn't push the right version, should be good now.

@carlosmn
Copy link
Member Author

Oh dang; python3 doesn't work on my desktop atm, I'll see about the python 3 issues.

This requires fairly little work on the pygit2 side to kick off all the
searching on the libgit2 side, so it's a fairly good candidate.

This changes the return value for the commit ids to Oid instead of
strings, which is what we generally try to return.
@carlosmn
Copy link
Member Author

Turns out I had my strings and bytes mixed up. All's good now.

@jdavid jdavid merged commit cd08425 into libgit2:master Sep 1, 2014
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.

2 participants