-
Notifications
You must be signed in to change notification settings - Fork 1.3k
chore: upgrade native libraries for 16KB page size compatibility #6445
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
base: main
Are you sure you want to change the base?
chore: upgrade native libraries for 16KB page size compatibility #6445
Conversation
@rohit9625 did you test it both on Android 15 and 16? Confirming because you'd an Android 16 device and had been testing on Android 15 but the PR description doesn't have those details. |
I have an Android 15 device but I only tested on Android 15 emulator with 16kb enabled. I couldn't find the option to enable 16kb page size on my physical device. However, I'll try uploading some images from my device without 16Kb as we need to make sure it works on both types of devices. |
✅ Generated APK variants! |
I experience a crash on this branch, which can not be reproduced on main:
|
Something strange is going on with Bookmarks>Pictures too, I am afraid. But I have not find a reproduction procedure yet.
|
I have this commit installed on my device and it crashes on tapping the target button on explore map: 6.0.2-debug-HEAD~f641e70. I'd reported a similar issue recently (user crash reports for v6.0.2). I've not installed the above APKs yet.
|
Yes, I'm also able to repro the crash on Explore for both this PR branch and main branch. Also, the problem 1 in this comment also seems to be happening on main. I can't see anything or any map at the Explore->Map, like on Nearby tab. |
Do you have any bookmarks when got the above crash? |
Explore>Map: Interesting, for me both problems do not happen on main, only on this branch. Bookmarks: Yes, I have some pictures bookmarked, and tapped one of them on the Bookmarks activity. The crash did not happen immediately. I will need to test more.
|
Okay, I'll also test this Bookmark flow.
Okay, so running on 4KB. That's great, we need to test on both page sizes and I'll also try to repro the issue on my physical device. Thanks :) |
How about breaking this down and figuring out which one is causing the crashes? If it's fresco, we can migrate. If MapLibre, we can look for a different version. Will help in narrowing down the problem. |
Description (required)
Fixes #6410
What changes did you make and why?
Changes made:
maplibre-native
to version 11.3.5 and AGP to 8.13.0fresco
library to latest version (3..6.0 and resolve build issueesTests performed (required)
Tests pending, we need to test:
Tested {build variant, e.g. ProdDebug} on {name of device or emulator} with API level {API level}.
Screenshots (for UI changes only)
none