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
Fix 317 #323
Fix 317 #323
Conversation
Looking into Travis |
@nmccready It's C stuff. Is there no way to test the memory leaks without it? |
@phated I think I know the problem. I already did this update for travis in the debug-fabulous project. Anyway travis made things more complicated and we need to tell it to explicitly install C libs for node. |
See travis.yml |
Ill fix it. |
Now looks like npm is failing to run or install on node 0.12 and 0.10. |
Don't install npm@latest... it no longer works on 0.10 - Either lock it at 4 or use the version it ships with. |
ugghhh.... I will keep digging or remove the memwatch-next stuff. |
0923ca5
to
da4cdb2
Compare
@phated finally fixed.. gave up on memwatch-next and all that. |
@nmccready not much for me to review now 😛 looks good to ship! |
Memory leak fixes and tests.