-
Notifications
You must be signed in to change notification settings - Fork 369
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
Device Armabi path error #47
Comments
This may be a general problem. I have seen crash reports in my application that may have been the same issue, so it would be worth fixing it in ReLinker rather than exposing the classes and forcing each application to solve it on their own. May I ask how are you planning to use the exposed interface? If the device claims that it supports only "armeabi", how will you recognize that it is wrong and "armeabi-v7a" is really supported? It could be really "armeabi"-only device and loading "armeabi-v7a" library will eventually crash with bad instruction. ReLinker currently uses |
there are some devices, i think with some new cpu Dual-core 1.5 GHz Krait |
maybe there can be a function which can supply preferred paths for 32 bit & 64 bit library in case of miss? |
This is a real pain point, but I think that the relinker API would be better if we added a way to override ABI detection, like an Exposing |
We are serving the through split apk based on abi on google play store as there are lot of jni libs for us. What we did was to look up in the path for which directory was present amongst these |
Does this issue have to do with Split APK? I just released version |
yes, the issue still remains. It is related to the jni folder name that android returns back. |
My vote remains firmly -1 on exposing internal relinker interfaces. We can address the issue either by adding an API to override ABI detection, or by adding additional smarts so that relinker tries to do the right thing. Option 1 is easy, but option 2 is hard... but good for users if it can be made to work. |
Will like to see some static API to be used for ABI detection in case of failure. |
why not try to copy so lib from base apk(the value of ApplicationInfo#sourceDir)? |
I agree with @benjamin-bader here. Would be nice if the library would do the right thing. Especially considering that the goal of this is to be a broad use library that makes loading native libs easy. Does anyone on this thread have the capacity at the moment to dig in and come up with the "doing the right thing" solution? Right now at Keepsafe we don't unfortunately. |
Hello guys, just FYI; twilio is using this library and it is causing some problems on apps on production. Devices we seen so far:
Both have android 7 Here is logs:
|
In some android devices we are getting list of abis as
"armabi" only however the jni files we have belong to armabi-v7a folder. This is causing crash in load at in some files.
Is it possible to expose the LibraryLoader interface? so we can specify the abi folders that our app supports
The text was updated successfully, but these errors were encountered: