Re: [Tarantool-patches] [PATCH luajit] ci: add LUAJIT_ENABLE_CHECKHOOK for exotic matrix

[Maxim Kokryashkin via Tarantool-patches <tarantool-patches@dev.tarantool.org>]

[-- Attachment #1: Type: text/plain, Size: 2045 bytes --]


Hi, Sergey!
Thanks for the patch!
LGTM
--
Best regards,
Maxim Kokryashkin
 
 
> 
>>This commit adds the build with the aforementioned option to test
>>on x86_64 and arm64 architectures to exotic build matrix.
>>---
>>
>>Branch:  https://github.com/tarantool/luajit/tree/skaplun/gh-noticket-ci-checkhook
>>Workflow example:  https://github.com/tarantool/luajit/actions/runs/4101136217
>>
>> .github/workflows/exotic-builds-testing.yml | 15 +++++++++++----
>> 1 file changed, 11 insertions(+), 4 deletions(-)
>>
>>diff --git a/.github/workflows/exotic-builds-testing.yml b/.github/workflows/exotic-builds-testing.yml
>>index cd0c14d7..aaf4f9f6 100644
>>--- a/.github/workflows/exotic-builds-testing.yml
>>+++ b/.github/workflows/exotic-builds-testing.yml
>>@@ -29,13 +29,14 @@ concurrency:
>>   cancel-in-progress: true
>> 
>> jobs:
>>- test-x86_64-exotic:
>>+ test-exotic:
>>     strategy:
>>       fail-fast: false
>>       matrix:
>>         BUILDTYPE: [Debug, Release]
>>+ ARCH: [ARM64, x86_64]
>>         GC64: [ON, OFF]
>>- FLAVOR: [dualnum]
>>+ FLAVOR: [dualnum, checkhook]
>>         include:
>>           - BUILDTYPE: Debug
>>             CMAKEFLAGS: -DCMAKE_BUILD_TYPE=Debug -DLUA_USE_ASSERT=ON -DLUA_USE_APICHECK=ON
>>@@ -43,10 +44,16 @@ jobs:
>>             CMAKEFLAGS: -DCMAKE_BUILD_TYPE=RelWithDebInfo
>>           - FLAVOR: dualnum
>>             FLAVORFLAGS: -DLUAJIT_NUMMODE=2
>>- runs-on: [self-hosted, regular, Linux, x86_64]
>>+ - FLAVOR: checkhook
>>+ FLAVORFLAGS: -DLUAJIT_ENABLE_CHECKHOOK=ON
>>+ exclude:
>>+ # DUALNUM is default for ARM64, no need for additional testing.
>>+ - FLAVOR: dualnum
>>+ ARCH: ARM64
>>+ runs-on: [self-hosted, regular, Linux, '${{ matrix.ARCH }}']
>>     name: >
>>       LuaJIT ${{ matrix.FLAVOR }}
>>- (Linux/x86_64)
>>+ (Linux/${{ matrix.ARCH }})
>>       ${{ matrix.BUILDTYPE }}
>>       GC64:${{ matrix.GC64 }}
>>     steps:
>>--
>>2.34.1
> 

[-- Attachment #2: Type: text/html, Size: 3405 bytes --]