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

📟 131 lua scripts #48

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from
Open

Conversation

landryarki
Copy link
Collaborator

This pull request integrates Lua scripting capabilities into the project by adding the lua-cpp library and updating various components to support Lua scripting. The most important changes include adding Lua dependencies, updating build scripts, and modifying components to interact with Lua.

Integration of Lua scripting:

  • Added lua-cpp submodule in .gitmodules and corresponding path (src/lua) and URL.
  • Updated conanfile.py to include lua-cpp/1.0.0 as a requirement.
  • Modified release.sh to handle Lua-related build and upload processes. [1] [2]

Build script updates:

  • Updated CMakeLists.txt files in src/common, src/engine, src/graphics, and src/physics to find and link lua-cpp and Lua libraries. [1] [2] [3] [4]

Component modifications:

  • Enhanced Transform and UIText components to implement LuaMetaObject for Lua scripting support. [1] [2] [3] [4] [5] [6] [7] [8]

New LuaScriptComponent:

  • Added LuaScriptComponent to manage Lua scripts, including its header and implementation files. [1] [2]

Engine modifications:

  • Updated Engine.cpp to register LuaScriptComponent and handle null components cleanup. [1] [2] [3]

@landryarki landryarki self-assigned this Nov 3, 2024
Copy link

linear bot commented Nov 3, 2024

EPI-131 Lua component

@Marius-P1 Marius-P1 force-pushed the feature/epi-131-lua-component branch 2 times, most recently from 59277f1 to 62676b4 Compare November 3, 2024 21:56
@Marius-P1 Marius-P1 force-pushed the feature/epi-131-lua-component branch from 62676b4 to 72a514d Compare November 3, 2024 22:01
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