Skip to content

Add MQ LuaRocks package repair#8

Closed
Teichoui wants to merge 1 commit into
mainfrom
codex/issue-19-luarocks-bad-version-fix
Closed

Add MQ LuaRocks package repair#8
Teichoui wants to merge 1 commit into
mainfrom
codex/issue-19-luarocks-bad-version-fix

Conversation

@Teichoui
Copy link
Copy Markdown
Owner

@Teichoui Teichoui commented May 7, 2026

Summary

Adds RedFetch support for checking and repairing common MacroQuest LuaRocks packages after updates, matching MQ PackageMan's LuaRocks tree and versioned repository behavior.

Why

Some Lua scripts depend on common rocks such as lsqlite3 and luafilesystem. RedFetch should be able to verify the common packages exist on the user's MQ install and install missing packages using the same MQ LuaRocks layout PackageMan uses.

Details

  • Adds redfetch lua fix to verify/install common MQ Lua packages.
  • Runs the same check after successful watched updates when a MacroQuest path is configured.
  • Locates luarocks.exe in the MQ root.
  • Uses the selected MQ modules/<jitVersion>/luarocks tree.
  • Uses the matching versioned MQ LuaRocks repository, e.g. https://luarocks.macroquest.org/2.1.1774638290/.
  • Keeps behavior aligned with MQ PackageMan rather than pinning a separate rock version.

Validation

  • python -m pytest tests/test_lua_packages.py
  • Verified in local MQ testing that PackageMan installed lsqlite3 0.9.6-1 and Bazaar loaded successfully.

@Teichoui Teichoui force-pushed the codex/issue-19-luarocks-bad-version-fix branch from b58d388 to 5838fe2 Compare May 7, 2026 01:44
@Teichoui Teichoui changed the title [codex] Add MQ LuaRocks package repair Add MQ LuaRocks package repair May 7, 2026
@Teichoui
Copy link
Copy Markdown
Owner Author

Teichoui commented May 7, 2026

Closing as a duplicate. The cleaned final LuaRocks repair work is now combined into PR #5.

@Teichoui Teichoui closed this May 7, 2026
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