-
Notifications
You must be signed in to change notification settings - Fork 67
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
undefined symbol: crypto_core_ed25519_scalarbytes #93
Comments
Hmm, this is strange. Our canary setup has not reported any errors and we cannot reproduce: https://travis-ci.org/sodium-friends/sodium-native-citgm/jobs/494464274 Are we correct in assuming you're using latest Node (11.x)? That's what I guessed from node-67 (ABI 67) |
Also can you share the output of |
Yes, I'm using 11.9 but I tried different node versions, and the result was the same :(
|
@jfr3000 It all seems normal to me. You can try going to the |
Did that, and same result 🤷♀️ @regular is having the same issue on his machine. Guess I'll stick with the workaround for now... |
I can also reproduce this with Electron 2 (ABI 57, same as Node 8.0.0). Here's what I'm seeing:
|
@christianbundy can you please run lld like above and report back? Also OS details would be helpful. We have a canary that test every night and have done extensive manual testing (we’re working on an electron app on macOS and Linux) and have not had these issues, so we suspect that something is wrong with your dynamic linker perhaps |
Sorry for the noise, I've tried reinstalling with |
@christianbundy you machine is probably alright. I am having this problem since a month or so, on both Linux and OSX. It happens so often that I've learned to live with it and my zsh autocompletes |
@regular we have a hard time reproducing, does this happen to you regularly with 2.3.0? |
@regular Can you please provide details of your OS and the ldd output so we can debug further? |
@regular @jfr3000 Are you running sodium-native through electron? Just fell over this: electron-userland/electron-builder#3590 |
I know what is going on now thanks to @andrewosh. Two different versions of libsodium are trying to be loaded, both compatible but one has a few more features than the other. It is something I've been working on for a while but never saw having practical relevance. Will try to come up with a fix for the new release |
Oh look, a scuttlebutt party. I'm hitting this bug with |
@mixmix The above fix I thought I'd found, I couldn't get to reproduce. I have the test case here: https://github.com/emilbayes/sodium-symbol-bug. Maybe someone could provide a complete |
Here's a repro that works on (at least) ubuntu and windows: https://github.com/vweevers/sodium-symbol-bug/tree/npm-aliases. What may help (probably not for this specific issue but you should do it anyway for mac), is adding the following to
To make symbols hidden. Because the problem is, on mac, See also nodejs/node-gyp#1891 and https://github.com/vweevers/about-native-modules |
Update: doesn't happen on mac, only ubuntu and windows. And only when using prebuilt binaries. Here's an alternative way to reproduce, that runs the tests of
|
I think this is fixed, if not reopen |
Hi! Could you link the commit or PR fixing this? I only notice #1046 mentioned, but that one was closed with no merge. |
This happens using the linux-x64/node-67.node prebuild. 2.2.6 works for me, so I guess this was introduced very recently? Any suggestions?
The text was updated successfully, but these errors were encountered: