-
Notifications
You must be signed in to change notification settings - Fork 332
error loading sodium bindings: libsodium.so.23 #1045
Comments
fixes #1045 and (theoretically) reduces build size - sodium-native is the only dependency using prebuilds - the only node binding we need is `electron-69.node` - where `$(electron --abi) == 69` - we also need the `libsodium.so.23` shared library - but we must copy this into the root directory of the build (outside the compiled binary file)
Any chance this was resolved by #1040? |
given my understanding, i don't think it would be, but i can check later. do you not see the same error message? |
No, I'm on Arch Linux and I've got something funky going on where AppImages work for me even though they throw errors on Ubuntu machines (both Mix and Piet had the same errors). The story so far:
I'd love to get rid of irrelevant prebuilds, but I want to make super sure it's not just working on my machine and breaking on Ubuntu. |
fixes #1045 and (theoretically) reduces build size - sodium-native is the only dependency using prebuilds - the only node binding we need is `electron-69.node` - where `$(electron --abi) == 69` - we also need the `libsodium.so.23` shared library - but we must copy this into the root directory of the build (outside the compiled binary file)
Btw, if you get a chance I'm super curious to know whether this is fixed in I really appreciate the tinkering you're doing with getting the build smaller, just know that if there are problems it's not you, and even though the docs seem authoritative I'm about 90% sure there's a bug in electron-builder that's borking the |
Just tested Ubuntu and it looks like this is working again. Woo! I'm going to remove the "blocking release" tag, is this good to close now? |
yes, this does work on the latest release! so i'll close this, but leave #1046 open (maybe rename) to focus on how we minimize the builds. and agree, "works on my machine" is not enough for a release, glad you were able to find something that works to release 😺 |
Woo! Happy this works, I'm excited to see the future of #1046. ❤️ |
when running latest
beta.2
, i get the following warning in the console:after doing a build locally and looking at
dist/linux-unpacked/
, i don't see anylibsodium.so.23
.i'll see if i can fix this, as i encountered this while working on #983, but turns out it affects the AppImage as well.
The text was updated successfully, but these errors were encountered: