Skip to content

Update libgit2 natives to 1.0.160#1349

Merged
carlosmn merged 1 commit into
masterfrom
ethomson/libgit2_update
Aug 5, 2016
Merged

Update libgit2 natives to 1.0.160#1349
carlosmn merged 1 commit into
masterfrom
ethomson/libgit2_update

Conversation

@ethomson

@ethomson ethomson commented Aug 4, 2016

Copy link
Copy Markdown
Member

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

@txdv

txdv commented Aug 5, 2016

Copy link
Copy Markdown
Contributor

How did I miss this...

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

Comment thread LibGit2Sharp/Core/NativeMethods.cs Outdated
/* 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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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 commented Aug 5, 2016

Copy link
Copy Markdown
Member Author

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

txdv commented Aug 5, 2016

Copy link
Copy Markdown
Contributor

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

carlosmn commented Aug 5, 2016

Copy link
Copy Markdown
Member

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.

Comment thread LibGit2Sharp/packages.config Outdated
<?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
Copy Markdown
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
Copy Markdown
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