-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
When building via App Bundle, Fresco fails to find "libimagepipeline.so" #2253
Comments
Does your error message looks like this one? #2049 |
Yes, roughly. However this issue while related I think more focuses on the
lack of support for Android App Bundle in either Fresco or SoLoader and
splitting binaries that way.
I could be mistaken, but this issue is reproducible on every device I've
tested, not just select brands.
And as per Fresco shipping guides, there is no note of the use case of
shipping via Android App Bundles, and we can't use the `splits` key as when
you use `bundle` the `splits` key is ignored.
…On Tue, 11 Dec 2018, 06:40 KimiChiu, ***@***.***> wrote:
Does your error message looks like this one? #2049
<#2049>
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#2253 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABUQnQ4cZ-sWkG5b72Ea81wiQjS02yjlks5u31NxgaJpZM4ZIsuf>
.
|
Hi @icerfish, Thank you for filing this issue with all the required details (maybe add a paste of the error if you have time). I can imagine that Fresco might not fully support app bundles and I do not think that we have ever tested the interplay. I will mark this as a "bug" and "help-wanted" hoping that this is something the open-source community can help us with. |
Hi @lambdapioneer, Here is the stacktrace:
It looks similar to issues that others are having. From looking around the codebase, it looks like this isn't an issue with Fresco itself, but more the SoLoader library. I will cross post the issue over to that repository, and workload allowing dedicate some time to try work out what is the issue exactly. |
Hi, I had the same issue when using app bundles, as described in #2049 (comment) Using Fresco 1.11 and setting |
I have found out an issue in SoLoader lib used by Fresco. I have prepared PR with a fix: facebook/SoLoader#26
|
The previous version of patched-soloader didn't work on pre-lollipop devices. I've fixed it. Use |
@nesterov-n thanks for great fix, any progress to integrate it into fresco? |
@nesterov-n i am also facing the same issue. please let me know when will it be integrated into fresco? |
Hi @theromis and @sailesh2 My PR to soloader lib is being reviewed now but soloader maintainer tells that there is no estimation of new soloader lib release. So It's impossible to estimate when Fresco will use this new version. If it's urgent you can use my patched version. We published bundle with fresco to production. Works well. |
Cannot figure out why isn't this workaround integrated into Fresco yet. |
We've landed the SoLoader fix. We'll release a new Fresco version soon once the SoLoader version has been released. |
@oprisnik what's the ETA for the releases? I get that it requires coordination of 2 teams but at least some rough number would help many developers decide if they should apply workaround above or wait for the release. |
SoLoader v0.6.0 has just been released and I've updated the Fresco dependency (6fc071d). We want to wait for the MIT licensed Bolts release (#2257). If that release takes longer than expected, we are going to skip it for now and release without that. In any case, the new version should be out in a couple of days. |
I don't want to be dull, but shouldn't fresco be released? Seems that this issue you pointed out hasn't received any activity for a while. |
We just released version 1.12.0 that includes the fixed SoLoader version. |
try add proguard Do not strip any method/class that is annotated with @DoNotStrip-keep @com.facebook.common.internal.DoNotStrip class * Do not strip any method/class that is annotated with @DoNotOptimize-keep @com.facebook.soloader.DoNotOptimize class * Keep native methods-keepclassmembers class * { -dontwarn okio.** |
@ProHzen Has this fixed the soloader crash ? |
Yes, I solved it. |
it is still in fresco 2.0.0, :-( , please suggest workaround i only have issue with nexus devices |
@ProHzen Hello, can you send me your confusion list?I tried it the way you did, but it didn't work. |
Description
When launching an Activity that uses Fresco, which has been generated via Android App Bundle, it fails to find "libimagepipeline.so" and crashes the application.
Reproduction
Solution
I originally thought this might have been an issue with minify, R8, or Proguard, but I disabled all of those, and still observed the same result when building via App Bundle.
I have tested other components of my app that also use native libraries, but they all operate as expected, only with Fresco struggling to load the respective binary.
A temporary, but not ideal solution I found is disabling splitting of APK's by
abi
using the following configuration, but including all the binaries results in a significantly larger APK size.Additional Information
The text was updated successfully, but these errors were encountered: