10000 Update libgit2 natives to 1.0.160 by ethomson · Pull Request #1349 · libgit2/libgit2sharp · GitHub
[go: up one dir, main page]

Skip to content

Update libgit2 natives to 1.0.160 #1349

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
Aug 5, 2016
Merged

Conversation

ethomson
Copy link
Member
@ethomson ethomson commented Aug 4, 2016

Update libgit2 to native version 1.0.160 (libgit2/libgit2@baa87df).

@txdv
Copy link
Contributor
txdv commented Aug 5, 2016

How did I miss this...

Looks, good, are you planning on exposing the new APIs in a different commit?

@@ -989,7 +976,7 @@ static NativeMethods()

/* Push network progress notification function */
internal delegate int git_push_transfer_progress(uint current, uint total, UIntPtr bytes, IntPtr payload);
internal delegate int git_packbuilder_progress(int stage, uint current, uint total, IntPtr payload);
internal delegate int git_packbuilder_progress(int stage, UInt32 current, UInt32 total, IntPtr payload);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this really needed? uint is always a synonym for System.UInt32

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Right, probably not. Thanks!

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

@ethomson
Copy link
Member Author
ethomson commented Aug 5, 2016

Looks, good, are you planning on exposing the new APIs in a different commit?

I don't have any plans to. Is there something in particular that you're interested in? Is that something that you'd want to tackle?

@txdv
Copy link
Contributor
txdv commented Aug 5, 2016

Yeah, I would like to tackle it(exposnig all the new APIs) beginning with (Set|Get)UserAgent . Actually, I already started doing that, but I ran against #1339 and kind of lost track...

@carlosmn
Copy link
Member
carlosmn commented Aug 5, 2016

Unless we've deprecated something, in which case we should immediately provide the new way of doing it, I'd rather not conflate the binaries update with exposing new functionality.

@@ -1,4 +1,4 @@
<?xml version="1.0" encoding="utf-8"?>
<packages>
<package id="LibGit2Sharp.NativeBinaries" version="1.0.157" targetFramework="net4" allowedVersions="[1.0.157]" />
<package id="LibGit2Sharp.NativeBinaries" version="1.0.160" targetFramework="net40" />
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this should still have a allowedVersions to stop it from trying to "update" to an incompatible version of the binaries.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

@ethomson ethomson force-pushed the ethomson/libgit2_update branch from 11381d5 to 88cf9a7 Compare August 5, 2016 14:16
@carlosmn carlosmn merged commit f12a329 into master Aug 5, 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.

3 participants
0