Skip to content

Add Android C++ support#24

Draft
ogoffart wants to merge 1 commit into
mainfrom
android-cpp-support
Draft

Add Android C++ support#24
ogoffart wants to merge 1 commit into
mainfrom
android-cpp-support

Conversation

@ogoffart

Copy link
Copy Markdown
Member

Build a shared library on Android (NDK auto-detected by Slint's CMake) and provide an extern "C" slint_main() entry point. Add a Gradle wrapper under android/ so users can run ./gradlew installDebug.

Build a shared library on Android (NDK auto-detected by Slint's CMake)
and provide an extern "C" slint_main() entry point. Add a Gradle
wrapper under android/ so users can run ./gradlew installDebug.
@ogoffart ogoffart requested a review from tronical May 29, 2026 10:16
@ogoffart

Copy link
Copy Markdown
Member Author

To be merged after the release.

That said... should we really merge it to master, or keep it in a branch and document it? or have it in a different repo?

Also, the UI doesn't look good because it doesn't take care of the safe area.

@tronical tronical left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's fine to have it in the same repo. No strong feeling though.

Comment thread src/main.cpp
});

ui->run();
return 0;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Won't this trigger static analysers?

@ogoffart ogoffart May 29, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TIL :)

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.

2 participants