search

iriscouch / follow

Very stable, very reliable, NodeJS CouchDB _changes follower
Apache License 2.0
393 stars 82 forks source link

log4js Dependency not listed. #34

Open nick-thompson opened 11 years ago

nick-thompson commented 11 years ago

Line 61 in ./lib/index.js conditionally requires a module named "log4js", which isn't specified as a dependency in the package.json. https://github.com/iriscouch/follow/blob/master/lib/index.js#L61

I found this trying to use browserify to render a client-side build of @dscape's Nano. Running browserify fails because the "log4js" dependency doesn't get installed here.

dscape commented 11 years ago

@substack is there a way to make browserify ignore this?

tellnes commented 11 years ago

@dscape you can put the require statment in its own file and replace that file with another with the browser field in package.json.

dscape commented 11 years ago

im not a comitter here