-
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
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. |
|
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. |
|
Great idea @RitikaPahwa4444 👍 Screen_recording_20251008_005115.mp4However, the app crashes when tapping center icon on the Explore->Map page and same crash on main as well. Are there some special conditions or maybe settings that might be causing this issue for your account, @nicolas-raoul? |
|
I just realized that Hacktoberfest is going on. What about adding the Hacktoberfest label to some issues, @nicolas-raoul? |
|
Sorry for pitching in but I had something in mind - Tip: keeping issues/PRs focussed not only helps trace bugs quickly but also gain better visibility :) @rohit9625 other community members can also share their insights if we have a new discussion thread/issue for Hacktoberfest. And if we're lucky, that thread can act as a motivator for potential contributors too ✨ |
|
We have no choice but to migrate to 16KB. So, how about merging this PR, and then creating new issues for the various issues we notice? (with hacktoberfest label unless somehow impossible to fix by an outsider) |
|
I tested a bit and only the target icon resulted in a crash, that too was a known issue. The plan sounds good to me. |
|
Sounds good, @nicolas-raoul. Well, did you test this PR from the above-generated APKs or by building the app on your pc? I doubt if any of the APKs have minification enabled, which can shrink some of the code that may be causing the issues you encountered. |
|
✅ Generated APK variants! |
|
I always build, using ./gradlew from command line, not adding any special settings I believe. 🙂 |

Description (required)
Fixes #6410
What changes did you make and why?
Changes made:
maplibre-nativeto version 11.3.5 and AGP to 8.13.0frescolibrary 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