Skip to content

Move AXGameFinalSetup.cmake to cpp and lua templates because of differences in code #2557

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

Merged
merged 2 commits into from
May 30, 2025

Conversation

rh101
Copy link
Contributor

@rh101 rh101 commented May 30, 2025

Describe your changes

An important line in the CMakeLists.txt that was different between cpp and lua projects was completely missed by me in PR #2426, so this fixes the issue.

The cpp project calls ax_link_cpp_prebuilt, but the lua project should be calling ax_link_lua_prebuilt.

Also added set(_AX_USE_PREBUILT FALSE) to the lua CMakeLists.txt.

Issue ticket number and link

#2556

Checklist before requesting a review

For each PR

  • Add Copyright if it missed:
    - "Copyright (c) 2019-present Axmol Engine contributors (see AUTHORS.md)."

  • I have performed a self-review of my code.

    Optional:

    • I have checked readme and add important infos to this PR.
    • I have added/adapted some tests too.

For core/new feature PR

  • I have checked readme and add important infos to this PR.
  • I have added thorough tests.

@halx99 halx99 added this to the 2.6.1 milestone May 30, 2025
@halx99 halx99 added the enhancement New feature or request label May 30, 2025
@halx99 halx99 merged commit 8f018cd into axmolengine:dev May 30, 2025
15 checks passed
@rh101 rh101 deleted the lua-prebuilt-fix branch May 30, 2025 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[LUA] Unresolved external symbol while using prebuilts workflow (lua project link error)
2 participants