Skip to content
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

ci: Add official cmake Android definitions #3632

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

leamas
Copy link
Collaborator

@leamas leamas commented Feb 7, 2024

Adding the official definitions for a cmake Android NDK cross build. Overall, this should simplify things over time. Notably, it makes the cmake ANDROID variable defined. This is the "official" way to determine an Android build and makes it possible to detect an Android build without resorting to the OpenCPN specific QT_ANDROID

Definitions copied from shipdriver where they have been used for long.

Merging this PR will need updating the complete Android build chain with similar definitions.

Adding the official definitions for a cmake Android NDK cross build.
Overall, this should simplify things over time. Notably, it makes
the cmake ANDROID variable defined. This is the "official" way to
determine an Android build and makes it possible to detect an Android
build without resorting to the OpenCPN specific QT_ANDROID

Definitions copied form shipdriver where they have been used for long.

Merging this PR will need updating the complete Android buld chain
with similar definitions.
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.

None yet

1 participant