-
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
Remove SoLoaderShim, use SoLoader #1989
Conversation
cefbd32
to
46afd54
Compare
@foghina has updated the pull request. |
46afd54
to
df9577d
Compare
@foghina has updated the pull request. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@oprisnik has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
df9577d
to
4a93e56
Compare
@foghina has updated the pull request. |
Rebased and using the latest SoLoader. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@foghina has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
Summary: SoLoader is more stable on older versions of Android. Switching to it should fix recurring issues that we get such as `UnsatisfiedLinkError` and so on. CI + run showcase app. Closes facebook/fresco#1989 Reviewed By: erikandre Differential Revision: D6556420 Pulled By: foghina fbshipit-source-id: ffc3faef6452294b4d781258c64ef57e08d107b5
Motivation (required)
SoLoader is more stable on older versions of Android. Switching to it should fix recurring issues that we get such as
UnsatisfiedLinkError
and so on.Test Plan (required)
CI + run showcase app.