Skip to content

iOS support #2411

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

Open
wants to merge 37 commits into
base: master
Choose a base branch
from
Open

iOS support #2411

wants to merge 37 commits into from

Conversation

joliver82
Copy link
Contributor

@joliver82 joliver82 commented Apr 25, 2025

  • Added jme3-ios-native library mostly based on old native iOS code prepared to work with both MOE and RoboVM (robovm is still WIP)
  • Modified jme3-ios adding buffer allocator and proper getPlatform method

Pending to add jme3-ios-native library to be uploaded to maven. Any help is appreciated

joliver82 and others added 30 commits May 17, 2018 23:07
… in the atlases instead of setting a predefined byte buffer

on disposal that made all atlases in the backend use the same buffer and generated rendering issues.
…d more images to the screens so it really uses more atlases
@codex128
Copy link
Contributor

You should probably write at least one test for this.

@stephengold
Copy link
Member

Pending to add jme3-ios-native library to be uploaded to maven. Any help is appreciated

If gradlew install installs the library to your Maven Local repository, then I expect gradlew release will upload it to the correct staging repo.

@joliver82
Copy link
Contributor Author

joliver82 commented Apr 27, 2025

Thanks for the comments

@codex128 , OK, I would write some, but... Which kind of tests do you want me to do? Also, how could I make them run either when compiling locally or in github actions if this code is intended to run only on iOS?

@stephengold , checked and the artifact is saved properly to mavenLocal. I had to make a small change so the jar is properly created when running "gradlew install". Thanks for the tip

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.

3 participants