-
Notifications
You must be signed in to change notification settings - Fork 30.1k
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
--enable-static fails due to multiple definition #13500
Comments
/cc @nodejs/diagnostics |
Same issue here. |
@deas did you find any workaround for this? |
The problem is with the cctest binary, it seems. You can probably work around it by just building node with |
@matiwinnetou Did not try very hard. Ended up assembling the final linking command manually. |
On a related note, there also seems to be a linking problem on win32 that might be of concern to this issue. I am building node from the checked out v7.9.0 git tag with
When I run exactly the same build from the node v7.4.0 tag everything links & works fine. |
Currently when building with --enabled-static the cctest target will include libraries to be linked regardless. This commit adds a condition to only add the libraries when dynamically linking. Fixes: nodejs#13500
ends up
for me.
With v7.4.0, things are working.
The text was updated successfully, but these errors were encountered: