Skip to content

Fix missing headers for LuaJIT#5

Open
sewbacca wants to merge 1 commit intopocomane:masterfrom
sewbacca:fix-luajit-compilation
Open

Fix missing headers for LuaJIT#5
sewbacca wants to merge 1 commit intopocomane:masterfrom
sewbacca:fix-luajit-compilation

Conversation

@sewbacca
Copy link

Fixes #4 by including necessary headers.

@leso-kn
Copy link

leso-kn commented Mar 12, 2025

@pocomane I can confirm this fixes LuaJIT support!
Any chance we can get it merged and upload a new release to luarocks? :)

I've just submitted a new AlpineLinux package which uses luachild https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/81038

but it's targeting LuaJIT and the workaround for the missing #include-statements I came up with is quite bulky.

luachild is the best subprocess library I've found so far!

@sewbacca sewbacca force-pushed the fix-luajit-compilation branch from fe0645d to 460c7d6 Compare April 20, 2025 02:09
@sewbacca
Copy link
Author

Hey @pocomane! Whats your current availability to merge this? =) I've uploaded a temporary package including the fix in the mean time.

I'd be happy to help out, in case you would like an additional maintainer for luachild!

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.

Cannot build with LuaJIT

2 participants