Fix C module loading on windows

leafo / gh-actions-lua

GitHub action for Lua/LuaJIT

83 stars 24 forks source link

Fix C module loading on windows #38

Open leso-kn opened 1 year ago

leso-kn commented 1 year ago

This PR addresses #39 and introduces 2 changes to the windows plain lua build:

Installs luaxx.dll next to lua.exe (bin/) where it can be picked up at runtime by native libraries linked against it
Links lua.exe against luaxx.dll to avoid "multiple lua VMs" errors

hishamhm commented 8 months ago

@leafo I've tested this in the CI runs for luasystem at https://github.com/lunarmodules/luasystem/pull/17 and I confirm this works! Without this patch, the resulting binary can only run pure Lua code (which meant LuaRocks ran fine, but then the C-based packages built by it wouldn't load in the interpreter).