Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Fix compilation for systems != windows and crash in diff function#922

Merged
johnhaley81 merged 1 commit intonodegit:masternodegit/nodegit:masterfrom
viatsko:fix-libgit2-crashCopy head branch name to clipboard
Feb 25, 2016
Merged

Fix compilation for systems != windows and crash in diff function#922
johnhaley81 merged 1 commit intonodegit:masternodegit/nodegit:masterfrom
viatsko:fix-libgit2-crashCopy head branch name to clipboard

Conversation

@viatsko
Copy link
Contributor

@viatsko viatsko commented Feb 25, 2016

Hi,

Atom crashed because of atom/atom#10965
Here's the same fix we've used for git-utils atom/git-utils#61
Full story about this bug in libgit2 issues tracker here libgit2/libgit2#2586

Please review as this makes library unusable in a lot of use-cases. :(

@joshaber
Copy link
Collaborator

Yup, this one's a doozy.

@johnhaley81
Copy link
Collaborator

@joshaber can you confirm this builds for electron please?Then we can do another bug fix release.

@viatsko thank you so much for this PR!!

@joshaber
Copy link
Collaborator

@johnhaley81 Verified builds on 0.36.8. 👍

@johnhaley81
Copy link
Collaborator

I'll get a release out. Thanks @viatsko @joshaber

johnhaley81 added a commit that referenced this pull request Feb 25, 2016
Fix compilation for systems != windows and crash in diff function
@johnhaley81 johnhaley81 merged commit 94834cb into nodegit:master Feb 25, 2016
@nathansobo
Copy link

❤️ ✨

@viatsko viatsko deleted the fix-libgit2-crash branch February 26, 2016 10:32
@maxbrunsfeld
Copy link

❤️

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.

5 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.