Skip to content

Android support#203

Draft
Murmele wants to merge 215 commits into
maplibre:mainfrom
Murmele:android-support
Draft

Android support#203
Murmele wants to merge 215 commits into
maplibre:mainfrom
Murmele:android-support

Conversation

@Murmele

@Murmele Murmele commented May 27, 2026

Copy link
Copy Markdown
Collaborator

Launch Checklist

  • Http accessor
  • SQLite Database Access
  • Fix app launch when not debugging (For some reason the app starts only when flashing with xbuild but anymore afterwards)

fixes: #98

  • Confirm your changes do not include backports from Mapbox projects (unless with compliant license) - if you are not sure about this, please ask!
  • Briefly describe the changes in this PR.
  • Link to related issues.
  • Include before/after visuals or gifs if this PR includes visual changes.
  • Write tests for all new functionality.
  • Document any changes to public APIs.
  • Post benchmark scores.
  • Add an entry to CHANGELOG.md under the ## main section.

Turbo87 and others added 30 commits October 27, 2025 01:30
- Add Android target detection for aarch64
- Add Android-specific system library linking (android, log, EGL, GLESv3)
- Skip curl linking on Android (uses Android's HTTP stack)
- Support both OpenGL ES and Vulkan rendering backends on Android
Document Android support including:
- System requirements and setup
- Rendering backend options (OpenGL ES and Vulkan)
- Build instructions for cross-compilation
- Required system libraries and dependencies
- Integration guide for Android apps
- Troubleshooting common issues
Reason: previously the wrong callback was used
Reason: otherwise the static_assertion checks are wrong
Reason: the MapObserver Instance got deleted after the MapRenderer_new_with_observer call, because the variable which has the pointer to the MapObserver gets deleted and then also the references pointing on it are invalid
Reason: unintentionally added. It does nothing
Reason: Otherwise we have a livetime problem if the function is not static. For Closures move must be used to capture all variables by value!
Reason: because it is important to set it
Reason: Using opaque objects is better so no unsafe conversion is required
Reason: less error prone and less unsafe code
@Murmele Murmele mentioned this pull request May 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Android support

4 participants