[Tarantool-patches] [PATCH luajit] build: introduce LUAJIT_ENABLE_CHECKHOOK option
    Sergey Kaplun 
    skaplun at tarantool.org
       
    Mon Oct  3 05:24:17 MSK 2022
    
    
  
Hi, Igor!
Thanks for the patch!
LGTM, except a single nit below.
Also, I suppose that the test will be added in the Tarantool patch with
version bump, as far as this option will be enabled by default for
Tarantool build.
Am I right?
On 02.10.22, Igor Munkin wrote:
> Originally there is nether a special option nor a variable to configure
> check for instrunction/line hooks for compiled code via the build system
Typo: s/check/the check/
> being used. We finally decided to use this feature in Tarantool, so for
> convenient managing LUAJIT_ENABLE_CHECKHOOK option is added to the root
> project CMakeLists.txt.
> 
> Needed for tarantool/tarantool#7762
> 
> Signed-off-by: Igor Munkin <imun at tarantool.org>
> ---
> 
> Issue: https://github.com/tarantool/tarantool/issues/7762
> Branch: https://github.com/tarantool/luajit/tree/imun/luajit-enable-checkhook
> CI: https://github.com/tarantool/luajit/commit/701de8c
> 
>  CMakeLists.txt | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/CMakeLists.txt b/CMakeLists.txt
> index 8b49f9d7..c870cce2 100644
> --- a/CMakeLists.txt
> +++ b/CMakeLists.txt
<snipped>
> -- 
> 2.34.0
> 
-- 
Best regards,
Sergey Kaplun
    
    
More information about the Tarantool-patches
mailing list