Skip to content

Update libjglios#2813

Merged
riccardobl merged 2 commits into
jMonkeyEngine:masterfrom
riccardobl:updateios
May 26, 2026
Merged

Update libjglios#2813
riccardobl merged 2 commits into
jMonkeyEngine:masterfrom
riccardobl:updateios

Conversation

@riccardobl
Copy link
Copy Markdown
Member

Changes:

  • support for custom app icon
  • support for assets bundling (they are now bundled separately from the executable)
  • libJGLIOS task renamed to simply iOS

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the iOS configuration and asset loading mechanism. It upgrades the libjglios dependency to version 0.4, migrates the Gradle configuration in jme3-ios-examples to use the new iOS block with configured assets and icons, and introduces IosBundleLocator to locate assets within the iOS application bundle. A review comment suggests optimizing the path validation in IosBundleLocator by using getPath() instead of redundant getCanonicalPath() calls to avoid unnecessary filesystem I/O.

Comment thread jme3-ios/src/main/java/com/jme3/asset/plugins/IosBundleLocator.java Outdated
….java

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@riccardobl riccardobl marked this pull request as ready for review May 26, 2026 07:21
@riccardobl riccardobl merged commit 16a28cc into jMonkeyEngine:master May 26, 2026
17 of 19 checks passed
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.

1 participant