-
Notifications
You must be signed in to change notification settings - Fork 0
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
sodium-native issues #5
Comments
Hi @staltz , I'm not aware about the internals of sodium, but one way of trying to debug what ends up on the apk would be to rename it to a zip file and looking into its contents (the Try to follow the require logic on your code. For example, what is your |
So, regarding the compilation of the If I copy-paste One thing I noticed is that this node module ( |
And regarding the structure of the apk,
But in reality it ends up nowhere because it seems to be deleted as a sibling of |
Hi @staltz , The issue is that only the Try to include |
@jaimecbernardo Thanks for the comment If I put the
While the ELF header for
|
Hi @staltz, It's possible that Something similar to:
Add it somewhere around here, I'd say. |
Thanks @jaimecbernardo, since that script would have been anyway specific to my app and the library sodium-native-nodejs-mobile, I ended up writing that copy script in a different way and in bash, but the end result is the same, and yes, it now works. I think I'm fine with this, it's not the best but it works, but I think the question that remains is how should node modules that provide dynamic libraries be included by nodejs-mobile, in general, and in an automated way. For instance, maybe nodejs-mobile's gradle script can run |
I started working on sodium-native supporting iOS, see this repo: https://github.com/staltz/sodium-native-nodejs-mobile I got quite far in the process of compiling sodium-native (and its subdep libsodium) to iOS arm64, but what I'm thinking now is what to do with this |
Hi @staltz I'm attempting to get After digging around I think it's some issue with the Did you encounter this when you were getting the library to work? And do you have any suggestions or potential solutions? I wasn't sure if it was more appropriate to comment here or open an issue on your repo so apologies if this is the wrong place. UPDATE: I attempted a to build on Android and I seem to be getting a similar error there as well. Here's the output:
|
I'm quite close to getting sodium-native (libsodium) working on nodejs-mobile, see https://github.com/staltz/sodium-native-nodejs-mobile but I'm hitting a runtime error:
I have tried to put
libsodium.so
innode_modules/sodium-native-nodejs-mobile/build/Release
but that has had no effect (with or withoutlibsodium.so
in that directory, I get the same runtime error).I'm running out of ideas, it could either be that
libsodium.so
should have been included insidebuild/Release/sodium.node
somehow, or it could be that I'm putting this file in the wrong directory. Any suggestions are welcome...The text was updated successfully, but these errors were encountered: