-
-
Notifications
You must be signed in to change notification settings - Fork 13
npm install fails with io.js #4
Comments
It also fails under Node 0.10.32 (OSX)
|
Agreed! Fails to install under OSX! |
This repo hasn't been updated in 2 years. @ralphtheninja should there be an indicator in the README that this repo is no longer maintained? |
@notduncansmith I think we should try to fix it instead. |
@ralphtheninja I have been working on this from my machine and should be able to get it working... I'll keep you posted. |
@bradserbu Cool! :) |
@ralphtheninja I'm pretty sure this will work if we get @rvagg to update the "rocksdb" NPMJS package to the latest version on github, v0.10.1, which builds on my machine. I've contacted him off line with the request. |
Added @ralphtheninja and @juliangruber to npm for |
No problem! Can you add me as a contributor as well on both the GitHub and NPMJS repositories. I plan to be using rocksDB and nodes extensively in the future and would love to contribute to these projects. Warm Regards, Sent from my iPhone
|
contributions first, access later |
Okay. I thought figuring out what was needed to fix this would count as a contribution. It may not have required coding changes, but it was time consuming. I will have to fork both projects, create a new npm repository for both the ‘level-rocksdb’ and ‘rocksdb’ packages, just to host a rebuild and wait until the pull requests are merged. -- On July 31, 2015 at 9:27:34 PM, Rod Vagg (notifications@github.com) wrote: contributions first, access later — |
if you're going to bring it up to date with the latest leveldown code, target this branch: Level/leveldown#195 which should be merged soon, also the package.json has a git ref but it can now just be nan |
ping This is still broken on nodejs 4.2.1 |
@ralphtheninja hey! do you know what the state of this is? Level/leveldown#195 has been merged, curious what next steps are. seeing this error with node 5.1.0 edit: works with 4.2.2 for me. my previous error was a node-gyp issue afaik |
No description provided.
The text was updated successfully, but these errors were encountered: