Skip to content

examples: added Android Vulkan example.#9399

Open
itsvks19 wants to merge 4 commits into
ocornut:masterfrom
itsvks19:master
Open

examples: added Android Vulkan example.#9399
itsvks19 wants to merge 4 commits into
ocornut:masterfrom
itsvks19:master

Conversation

@itsvks19
Copy link
Copy Markdown

No description provided.

@ocornut
Copy link
Copy Markdown
Owner

ocornut commented May 15, 2026

Hello,

  • This seems to have an unrelated changes to the Android backend.
  • Examples should match. Here the directory layout and files does not match example_android_opengl3. I'm not against evolving the layout or that one if there's a good reason, but they should match as much as possible.
  • Binary should not be committed into the repo (librairies, unnecessary icons if it would work without).

In am in principle 100% of an example of this kind to be added but I would need you to be ten times more careful about every changes.

- Moved Gradle build files, wrapper, and project settings into a new 'android' subdirectory.
- Relocated main.cpp and CMakeLists.txt to the example root directory.
- Simplified AndroidManifest.xml and removed redundant resource files (colors, themes, mipmaps).
- Updated CMake configuration to reflect the new file locations.
- Relocated JNI validation layers.
@itsvks19
Copy link
Copy Markdown
Author

I refactored to match the layout same as 'example_android_opengl3'.

Binary should not be committed into the repo

Vulkan's validation layer will not work without those binaries in Android.

@ocornut
Copy link
Copy Markdown
Owner

ocornut commented May 16, 2026

We should find another solution. Precompiled binaries are bounds to get updated, and their size is currently larger than the whole of the existing repository and history, so obviously it is unacceptable as is.

@itsvks19
Copy link
Copy Markdown
Author

itsvks19 commented May 16, 2026

I have a solution, I will download those binaries at build time and place them in their folder.

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