[Tarantool-patches] [PATCH luajit 2/3] ci: introduce workflow for exotic builds
Igor Munkin
imun at tarantool.org
Wed Feb 1 11:27:29 MSK 2023
Hi everyone,
On 30.01.23, Sergey Kaplun wrote:
<snipped>
> >
> > Shall we add the -DLUAJIT_ENABLE_CHECKHOOK and -DLUAJIT_SMART_STRINGS=1 since they are in Tarantool by default?
>
> -DLUAJIT_SMART_STRINGS=1 is set by default, so it's OK not to set it
> mannually.
>
> OTOH, -DLUAJIT_ENABLE_CHECKHOOK is not set by default.
> I prefer not to set it for this particular workflow, because I plan to
> use it for very specific builds (even impossible for Tarantool) for
> example with -DLUAJIT_DISABLE_JIT=ON, -DLUAJIT_DISABLE_FFI=ON flags.
> Also, this flag is not set for default LuaJIT testing, so I tried to
> make those workflows look similar.
> But if we want to add this flag for default build or testing we should
> add it for this workflow too.
> CC-ed Igor to hear his opinion.
I'm totally for adding such workflows, but I suggest to do it in a
separate patch. Sergey, this is on you.
>
> >
<snipped>
>
> --
> Best regards,
> Sergey Kaplun
--
Best regards,
IM
More information about the Tarantool-patches
mailing list