FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Bump libgit2 to 43275f512ef888a14c43bde8007251ce595be32a + unmerged proxy fixes by srajko · Pull Request #1223 · nodegit/nodegit · GitHub

Repository navigation

Bump libgit2 to 43275f512ef888a14c43bde8007251ce595be32a + unmerged proxy fixes - #1223

Merged
johnhaley81 merged 6 commits into
nodegit:masterfrom
srajko:bump-libgit
Feb 28, 2017
Merged

johnhaley81 merged 6 commits into
nodegit:masterfrom
srajko:bump-libgit

Conversation

srajko commented Feb 14, 2017

Copy link
Copy Markdown
Collaborator

No description provided.

rcjsuen commented Feb 15, 2017

Copy link
Copy Markdown
Member

Bumping libgit2 to master would also fix #1220. Keep in mind though that things are a little fishy there since the test suite crashes but the builds are green...

srajko commented Feb 16, 2017

Copy link
Copy Markdown
Collaborator Author

@rcjsuen in which builds are you seeing the tests crash?

rcjsuen commented Feb 16, 2017 •
edited
Loading

Copy link
Copy Markdown
Member

@srajko Take a look at the AppVeyor and Travis CI builds that got spawned off for #1220. While AppVeyor is correctly red, Travis CI is green even though the test suite has segfaulted... :(

srajko commented Feb 16, 2017

Copy link
Copy Markdown
Collaborator Author

That is indeed fishy :-) Thanks for pointing that out, definitely something we should resolve.

tommoor commented Feb 17, 2017 •
edited
Loading

Copy link
Copy Markdown

0.17.0 is also crashing our test suites, I'm not sure why as karma disconnects with no result - subscribing to this in the hope that this will fix our woes when merged!

srajko changed the title Bump libgit2 to master + unmerged proxy fixes Bump libgit2 to 43275f512ef888a14c43bde8007251ce595be32a + unmerged proxy fixes Feb 24, 2017

Copy link
Copy Markdown
Collaborator

@srajko this needs a rebase now :(

Copy link
Copy Markdown
Collaborator

Well, everything is green except for the 32-bit linux builds which aren't really related to this PR and are fixed in #1241 so I'll admin merge this. Thanks @srajko

johnhaley81 merged commit 6b1f73b into nodegit:master Feb 28, 2017
johnhaley81 deleted the bump-libgit branch February 28, 2017 15:11
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

4 participants


Back | FazBrowse Home | New Git URL