Skip to content

Remove obsolete field. #1339

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

Merged
merged 1 commit into from
Jul 29, 2016
Merged

Conversation

txdv
Copy link
Contributor

@txdv txdv commented Jul 13, 2016

This field was removed from libgit2 a year and a half ago.

This is the libgit2 commit: libgit2/libgit2@659cf20#diff-7c9f3aae8ea34d171fe8f44fbedde372L140

This is the libgit2sharp commit: 4c5c088

This field was removed from libgit2 a year and a half ago.
@txdv
Copy link
Contributor Author

txdv commented Jul 13, 2016

This was exposed because I was fiddling with the git_fetch_options structure (it is included in the git_clone_options structure), making it bigger before this field.

@carlosmn
Copy link
Member

The calendar time doesn't really say much. Is this removed in the current version of libgit2 we target/bundle?

@txdv
Copy link
Contributor Author

txdv commented Jul 13, 2016

Yes, it used to be here, but now it is gone.

@txdv
Copy link
Contributor Author

txdv commented Jul 13, 2016

I linked master, but the commit at which NativeBinaries is targeted is missing the signature field too: https://github.com/libgit2/libgit2/blob/381caf56908ad9cdb5959850a331d7cf2f952e63/include/git2/clone.h#L136-L143

@txdv
Copy link
Contributor Author

txdv commented Jul 18, 2016

ping @carlosmn

@carlosmn carlosmn merged commit 3febf8d into libgit2:master Jul 29, 2016
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