[Tarantool-patches] [PATCH luajit 2/3] ci: introduce workflow for exotic builds
Sergey Kaplun
skaplun at tarantool.org
Wed Feb 1 11:52:56 MSK 2023
Hi, Igor!
Thanks for reply:)!
On 01.02.23, Igor Munkin wrote:
> 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.
OK, so I'll add exotic -DLUAJIT_ENBALE_CHECKHOOK job after this patch
will be merged to avoid rebases.
>
> >
> > >
>
> <snipped>
>
> >
> > --
> > Best regards,
> > Sergey Kaplun
>
> --
> Best regards,
> IM
--
Best regards,
Sergey Kaplun
More information about the Tarantool-patches
mailing list