-
Notifications
You must be signed in to change notification settings - Fork 628
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
Port to Node.JS 4.x #253
Comments
+1 |
Affected by this too +1 |
+1 |
So, can this issue be considered resolved as the PR has been merged? |
Is this issue closed? I tried with 0.2.29 and still getting error. |
@ghughal could you share the errors you are getting? |
Here's the error I'm getting. There are bunch of C/C++ error before this. I'm building this on SUSE Linux Enterprise Server 11 (x86_64)
|
@ghughal please share gcc version you have on the machine, if not latest update and give it a try. |
Right. I have failed once but after upgrading to g++ 4.8, the compilation was successful. I haven't particularly tried kafka-node but node-hashes. I suppose he reason should be the same.
|
We're been using this in a 4.x environment w/o issues. |
Noted here, this package is currently failing to build under Node.JS 4.x.
The upstream issue to track: nodejs/node#2798
It looks as though this is (at minimum) based on [email protected] being included in the package.json. [email protected] does compile correctly on Node.JS 4.x.
The text was updated successfully, but these errors were encountered: