WeaselGames / godot_luaAPI

Godot LuaAPI
https://luaapi.weaselgames.info
Other
348 stars 27 forks source link

Bug Report: Binding libraries does not work on Android release builds #205

Closed Connor-McCloskey closed 3 months ago

Connor-McCloskey commented 3 months ago

Describe the bug This seems to be similar in some ways to the reported bug where libraries are not binding on Mac platforms. I built an Android release with export templates and Android addon items I compiled myself from this repo and exported an experimental release build of my project. I encountered a similar issue where all of the libraries I bound in my initial setup of the VM from GDScript failed.

To Reproduce

  1. Create a project
  2. Ensure you have the necessary Android files for the LuaAPI plugin
  3. Install the Android build template for Godot
  4. Set up an Android Gradle build and configure it for release configuration
  5. Set up a test script for initializing a VM and pushing some libraries (for example, table and package) to Lua
  6. Observe in your test that the libraries are not bound

Expected behavior Bound libraries should be bound!

Screenshots If applicable, add screenshots to help explain your problem.

Environment (please complete the following information):

Connor-McCloskey commented 3 months ago

Update - I'd incorporated the changes from PR #196 which resulted in this behavior, which was also causing a seg fault on Android builds. Reverting the changes fixes these bugs, but doesn't point us toward a unified cross-platform solution for library binding.