Skip to content

Remove obsolete field.#1339

Merged
carlosmn merged 1 commit into
libgit2:masterfrom
txdv:git-clone-options-field
Jul 29, 2016
Merged

Remove obsolete field.#1339
carlosmn merged 1 commit into
libgit2:masterfrom
txdv:git-clone-options-field

Conversation

@txdv

@txdv txdv commented Jul 13, 2016

Copy link
Copy Markdown
Contributor

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

txdv commented Jul 13, 2016

Copy link
Copy Markdown
Contributor Author

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
Copy Markdown
Member

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

@txdv

txdv commented Jul 13, 2016

Copy link
Copy Markdown
Contributor Author

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

@txdv

txdv commented Jul 13, 2016

Copy link
Copy Markdown
Contributor Author

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

txdv commented Jul 18, 2016

Copy link
Copy Markdown
Contributor Author

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