E55B Master by kam8818 · Pull Request #1154 · nodegit/nodegit · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@kam8818
Copy link
@kam8818 kam8818 commented Oct 26, 2016

No description provided.

srajko and others added 30 commits March 23, 2016 16:20
Affects blob objects returned by:
git_blob_lookup
git_blob_lookup_prefix
It's possible that we will eventually get rid of the selfFreeing parameter, and just go by the type descriptor, but for now we are more comfortable with a smaller change.
the structs git_tree_entry, git_oid, and git_signature all represent structs which are owned by some other object. In order to simplify our memory model, we should always duplicate these 3 structs when we pull them out of libgit2. This detaches these types of structs from their owning object, such that when we delete a commit, none of the oids or signatures that we retrieved from that commit would cause bad memory access.
Call free on GC for object types
diff.blobToBuffer: Fix passing in Buffers instead of strings.
Switch from strdup to memcpy to faithfully copy all bytes and prevent a crash in some cases.
Add support for thread safety on async actions only
Add support for git_repository__cleanup
maxkorp and others added 29 commits September 14, 2016 15:53
Since we can't require it from node if it's compiled for electron, we should just assume success for now
fix up electron and nw.js docs
Don't run postbuild when we detect electron install
Compile libgit2 with GIT_SSH_MEMORY_CREDENTIALS defined to allow
sshKeyMemoryNew to work.

Fixes #760
Define GIT_SSH_MEMORY_CREDENTIALS for libgit2
Add "Path" to ssh variable names in tests for clarity
Revert "Use npm 2.15.11 with node 0.12"

This reverts commit 9503b5c.
Revert "Log npm-debug.log after failure"

This reverts commit 412ce63.
Remove ls attempt


Ls purported npm-debug.log location and ignore npm install failure


Attempt to cat npm-debug.log after install failure


Log npm-debug.log after failure


Use npm 2.15.11 with node 0.12
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.

0