tym icon indicating copy to clipboard operation
tym copied to clipboard

Published 20 hours ago • verified publisher icon endaaman

LuaJIT support

Open thalting opened this issue 2 years ago • 7 comments

TODO: turn this into a build flag.

thalting avatar Jun 17 '22 18:06 thalting

Thanks for the PR.

I think this should be the compile option to enbale by like --enable-luajit.

endaaman avatar Jun 20 '22 05:06 endaaman

how can I do this?

thalting avatar Jun 21 '22 04:06 thalting

What you have to do

  • Edit configure.ac, introduce --with-luajit option and define the flag that luajit is enabled or not.
  • Write LuaJIT-supported code along with the flag.
  • Fix Dockerfile or add new test suite for LuaJIT compilation.

endaaman avatar Jun 21 '22 06:06 endaaman

i don't know how to use autotools :/

thalting avatar Jun 23 '22 20:06 thalting

--enable-luajit by @capezotte

thalting avatar Aug 05 '22 04:08 thalting

--enable-luajit and USES_LUAJIT are defined but it seems LuaJIT is not actually used.

endaaman avatar Aug 06 '22 03:08 endaaman

foo

thalting avatar Aug 06 '22 15:08 thalting

merge?

ecesar88 avatar Sep 02 '22 01:09 ecesar88

I wanted the review fixed. Anyway, it is OK. I will fix it. Thanks for the PR!

endaaman avatar Sep 02 '22 05:09 endaaman