-
Notifications
You must be signed in to change notification settings - Fork 39
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
Fails to compile (on MacOS) with Node 10 #98
Comments
Hi, this is a problem with nan not updated to latest which is required to be compiled with Node10. I've tried a dirty hack by copying the whole zopfli code to the repo and it can be installed normally. I will create PR for this fix and hope the owner will have sometime to merge it. |
take back the submodule thing, in fact we just need to force it to build again then it should work: in package.json's
to
and it should work |
you can try it out here |
@chinhnguyen do you have your updated package published anywhere? |
I'm afraid not, currently I'm pointing directly to my own fork. |
Just hit that issue as well under Windows 8.1 + MSYS2. It is apparently caused by a behavior change in node 10 (issue #21063), which was addressed recently in To fix this issue one can alter
After that change running |
Awesome! Thanks!
|
Woo thanks @geonanorch you helpful Windows 8 user 😂 |
fixed in the new published v2.0.3 |
node: v10.9.0
node-gyp: v3.7.0
Works fine with node 8 and 9.
Some of the people in #87 might actually be experiencing this issue instead.
The text was updated successfully, but these errors were encountered: