Skip to content
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

Error installing jimp, 'node-zlib-backport error' #129

Closed
kobimx opened this issue Apr 26, 2016 · 15 comments
Closed

Error installing jimp, 'node-zlib-backport error' #129

kobimx opened this issue Apr 26, 2016 · 15 comments

Comments

@kobimx
Copy link

kobimx commented Apr 26, 2016

So, for some reason, out of the blue my nodejs refuses to work as it should. when i run npm insall jimp i get a node-gyp error on the module node-zlib-backport used by the dependency pngjs. it returns an error when compiling the source giving a `Users/mirko/.node-gyp/5.11.0/include/node/node_internals.h:6:10: fatal error: 'util-inl.h' file not found

include "util-inl.h"` error.

Heres the (almost) full terminal log. i know this might not be jimp's problem but i wonder if anyone experienced it before:

node-zlib-backport@0.11.15 install /Users/mirko/seriesrocketNotifierApp/node_modules/node-zlib-backport
> node-gyp rebuild

  CXX(target) Release/obj.target/zlib/src/node_zlib.o
In file included from ../src/node_zlib.cc:32:
/Users/mirko/.node-gyp/5.11.0/include/node/node_internals.h:6:10: fatal error: 'util-inl.h' file not found
#include "util-inl.h"
         ^
1 error generated.
make: *** [Release/obj.target/zlib/src/node_zlib.o] Error 1
gyp ERR! build error 
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:276:23)
gyp ERR! stack     at emitTwo (events.js:100:13)
gyp ERR! stack     at ChildProcess.emit (events.js:185:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:204:12)
gyp ERR! System Darwin 15.4.0
gyp ERR! command "/usr/local/bin/node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /Users/mirko/seriesrocketNotifierApp/node_modules/node-zlib-backport
gyp ERR! node -v v5.11.0
gyp ERR! node-gyp -v v3.3.1
gyp ERR! not ok 
npm WARN install:node-zlib-backport@0.11.15 node-zlib-backport@0.11.15 install: `node-gyp rebuild`
npm WARN install:node-zlib-backport@0.11.15 Exit status 1

Running on OS X with node 5.11.0 and npm 3.8.6

@oliver-moran
Copy link
Collaborator

Thanks! Looks like a problem with the latest version of pngjs. The previous version is still good. I'll do an emergency release tonight.

@malexdev
Copy link

@oliver-moran getting this as well, thanks so much for the quick response! 😀

@kobimx
Copy link
Author

kobimx commented Apr 26, 2016

Yeah, thanks for the super quick response @oliver-moran, hope it will not cause anymore problems

@oliver-moran
Copy link
Collaborator

OK. I did an emergency release just now. (Even though it broke the browserify version by doing so.)

Can you guys check that all is OK again? Thanks.

@rjw209
Copy link

rjw209 commented Apr 27, 2016

@comigor
Copy link
Contributor

comigor commented Apr 27, 2016

@lukeapage just updated pngjs to 2.3.1, which should fix our issue

@lukeapage
Copy link

Sorry about this - it is a bug in npm - I should be able to add an optional dependency without breaking peoples builds, but nevermind, the dependency is removed.

@manikmi
Copy link

manikmi commented Apr 27, 2016

It stopped working with my nodejs version. (I think node compatability is the reason) my nodejs version is 0.12.9. It gives the error with histogram method (ln 1169) in index.js for => notation.
Do we have a version which can work with node 0.12.9?

@malexdev
Copy link

@oliver-moran Works perfectly on Node 5.11 for me. Thanks again!

@rjw209
Copy link

rjw209 commented Apr 27, 2016

@oliver-moran per @manikmi and my comment above, there's an ES6 arrow function that breaks compatibility with older versions of node. @Igor1201 PR#130 includes a fix for this, but not sure when you plan to merge/release that work.

@comigor
Copy link
Contributor

comigor commented Apr 27, 2016

we should just fix this threading arrow problem on master, because my PR is supposed to do other things as well

@oliver-moran
Copy link
Collaborator

Just checking, any of these issues still presenting?

@malexdev
Copy link

Ever since I upgraded the project that uses this to node 5.11 I haven't been having any issues.

Alex

On Tue, Jul 26, 2016 at 4:53 PM -0500, "Oliver Moran" notifications@github.com wrote:

Just checking, any of these issues still presenting?


You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

@kobimx
Copy link
Author

kobimx commented Jul 26, 2016

Issue can be closed, it's safe to say that it's fixed now 👍

@oliver-moran
Copy link
Collaborator

Happy days! Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants