leveldown dep missing? #6

Closed
brycebaril opened this Issue May 9, 2013 · 2 comments

Comments

Projects
None yet
3 participants
Owner

brycebaril commented May 9, 2013

Not sure if this is something missing from the dependencies in this or maybe just my browserify inexperience, but it seems to want leveldown even if it isn't really using it.

Installing leveldown made browserify happy. If it's not just user error on my part it might help to put it in the package.json as a dep.

Owner

rvagg commented May 10, 2013

If this is from using LevelUP along with level.js, then we're getting there: rvagg/node-levelup#129 (comment)
0.9 should see a proper decoupling from LevelDOWN that will allow a use-case like level.js to work better. In the meantime you should perhaps tell browserify explicitly to ignore leveldown.

Owner

brycebaril commented May 10, 2013

Yup, that's it -- I'm working off of the LevelUP/level-js example in the code here.

I can't seem to find the invocation to make browserify happy ignoring leveldown, but it's not really a big deal to just have leveldown installed for now.

Thanks for making this & helping me out

@maxogden maxogden closed this Feb 24, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment