-
-
Notifications
You must be signed in to change notification settings - Fork 88
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
Argon2 build fails on GCC 4.8 #117
Comments
+1 |
It is actually a GCC 4.8 issue. Try clang or GCC 5+ |
I had a similar issue, eventually after making sure I had the following packages installed it worked: |
Getting same issue with gcc v4.9.2. Just using a Dockerfile with |
I had the option to install a newer GCC on CentOS 7. It is easy using Software Collections: https://www.softwarecollections.org/en/scls/rhscl/devtoolset-7/ |
@kingjerod do you get the same with |
On one of my projects this was fixed by upgrading to node:8.9-stretch. However, I have a number of images based on OS like Ubuntu 16.04 (or 14.04) that fail. Upgrading gcc is a bit annoying if it's only to use argon2 17... I've had to downgrade to 16.2 on a couple more. Would love to see gcc support for 4.8 return... otherwise I'll be stuck for a while. |
@ranisalt I have not tried it with alpine, but in my experience alpine rarely works for any packages that need to compile or use node-gyp. I switched from alpine to non-alpine to get bcrypt to work. Just less headaches at the cost of 10x image size. |
@ranisalt Completely fails on node:8.9-alpine due to lack of Python:
|
Install the required build tools install argon2 and remove the build tools.
|
There is also similar issue for windows #109 |
That's not the same issue. Here, it is syntax parsing problem with a very old version of GCC, on the other issue it's a compiler library shipped corrupted (if I understood it correctly). IOW, this one is "fixable" by downgrading syntax, the other one is not and requires user action. |
Fixed it, but you really should update your compilers. Use docker if you must use Ubuntu 14.04 to access newer versions. (or ditch Ubuntu for once) |
The official Docker Node images are based off Debian Jessie and it wouldn't build on them either. |
but ubuntu is the coolest ;) |
Isn't Debian Jessie on GCC 4.9? I got it working here hahaha but anyways, it's fixed. |
Steps to reproduce
Expected behaviour
The package is installed correctly, the build of the C++ argon2 binary does not fail. The build did not fail with argon2 v0.16.2
Actual behaviour
The build of argon2 v0.17.2 binary fails with following errors:
Environment
Operating system:
CentOS 7 3.10.0-693.17.1.el7.x86_64
Node version:
8.9.4
Compiler version:
The text was updated successfully, but these errors were encountered: