Skip to content
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

Add regression test for libgit2 #4102 #1220

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rcjsuen
Copy link
Member

@rcjsuen rcjsuen commented Feb 9, 2017

If Revwalk.SORT.TOPOLOGICAL is used in a revwalk, Node will crash. This is caused by libgit #4102.

This is blocking the Orion project from upgrading to NodeGit 0.17.0.

Using a Revwalk.SORT.TOPOLOGICAL in a revwalk is causing libgit2 to
crash. This regression test should help ensure this bug is covered
by the test suite in the future.
@rcjsuen
Copy link
Member Author

rcjsuen commented Feb 9, 2017

The test is crashing early as expected (since the Revwalk tests are the last one it executes) but the 0 exit code is flagging the builds as green on Travis. Hm...

The command "if [ $TARGET_ARCH == "x64" ]; then if [ -z "$TRAVIS_TAG" ] && [ $TRAVIS_OS_NAME == "linux" ] && [ $NODE_VERSION == "6" ]; then npm test && npm run cov && npm run coveralls; else npm test; fi else echo "Not running tests because the binary is not built for 64-bit systems"; fi" exited with 0.

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.

1 participant