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

Fix android build #45

Merged
merged 16 commits into from
Feb 28, 2022
Merged

Fix android build #45

merged 16 commits into from
Feb 28, 2022

Conversation

JessyDL
Copy link
Owner

@JessyDL JessyDL commented Feb 24, 2022

added build for android, fixed various compile errors related to android platform

Removed the following CMake settings:

  • PE_PCH: was unused and untested for a while now
  • PE_SOURCE_ONLY: lost its raison d'être a while ago

Added new define:
SURFACE_ANDROID, due to the different nature of the android surface this felt appropriate.

std::execution was wrapped in psl::ecs::execution (only place this was used). Android's NDK doesn't have it defined yet. Now comes with psl::ecs::execution::no_exec for when it's unsupported.

core::os::context added, empty on non-android systems, but on android has info when the app goes to the background (as well as window info). Considering to move to core::os::surface, but not sure yet.

Currently android builds are set up in the following way:
invoke python3 tools/android.py will set up the project files in builds/android. From there you can invoke ./gradlew bundle in builds/android. Output will appear in the expected location for an android project.

Alternatively setting PE_PLATFORM in cmake will generate the projects as well (internally by calling the previous script), two utility targets will be made android_paradigm_generate and android_paradigm_build (latter has dependency on the former)

@JessyDL JessyDL added this to the 0.2.0 milestone Feb 24, 2022
@JessyDL JessyDL mentioned this pull request Feb 24, 2022
2 tasks
@JessyDL JessyDL force-pushed the fix/android-support branch from dd1c891 to 704b30d Compare February 26, 2022 21:57
@JessyDL JessyDL force-pushed the fix/android-support branch 3 times, most recently from 117c937 to af36384 Compare February 28, 2022 14:11
@JessyDL JessyDL force-pushed the fix/android-support branch from af36384 to 1ecca14 Compare February 28, 2022 14:22
@JessyDL JessyDL merged commit 41e4bf8 into develop Feb 28, 2022
@JessyDL JessyDL deleted the fix/android-support branch February 28, 2022 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant