blackbeam / poppler-simple

A simple javascript interface to poppler library
Apache License 2.0
37 stars 14 forks source link

Failing node-gyp configure/rebuild results in successful installation #13

Closed jessecstewart closed 8 years ago

jessecstewart commented 8 years ago

When running npm install poppler-simple, if the node-gyp compilation fails the npm install still succeeds. This is undesirable because it is hard to tell what went wrong (especially if it fails as part of a lengthy npm install). The problem appears to be these few lines -- https://github.com/blackbeam/poppler-simple/blob/master/package.json#L14 Where exit 0 should probably be exit 1.

blackbeam commented 8 years ago

Thanks!