[Tarantool-patches] [PATCH luajit v2 1/5] ci: clean up workflow for exotic builds
Igor Munkin
imun at tarantool.org
Fri Jul 21 11:12:04 MSK 2023
Unfortunately, the same mistake, that was introduced in the very first
commit f877b7691d9eae73d64c63ee3770089569915459 ("ci: introduce GitHub
Actions") and was fixed in the main testing pipeline in scope of the
commit 524616a35c1267cb9ca260b8456b71dde1881e22 ("ci: remove GC64 matrix
entries for ARM64 workflows"), was made in the new CI pipeline added in
scope of the commit 32ac0cadcba58cd69058084c88100fdcc7b7dccf ("ci: add
ARM64 architecture to exotic testing").
Specifically, CMake option LUAJIT_ENABLE_GC64 changes nothing for ARM64
LuaJIT builds (i.e. LJ_GC64 is defined on ARM64 platforms despite the
value of LUAJIT_ENABLE_GC64). As a result GC64 entry is removed from
all ARM64 flavors in exotic builds testing routine.
Besides, all flavor labels are sorted alphabetically.
Signed-off-by: Igor Munkin <imun at tarantool.org>
---
.github/workflows/exotic-builds-testing.yml | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/.github/workflows/exotic-builds-testing.yml b/.github/workflows/exotic-builds-testing.yml
index 318fb4dd..7c57d238 100644
--- a/.github/workflows/exotic-builds-testing.yml
+++ b/.github/workflows/exotic-builds-testing.yml
@@ -34,7 +34,7 @@ jobs:
BUILDTYPE: [Debug, Release]
ARCH: [ARM64, x86_64]
GC64: [ON, OFF]
- FLAVOR: [dualnum, checkhook, nojit, gdbjit]
+ FLAVOR: [checkhook, dualnum, gdbjit, nojit]
include:
- BUILDTYPE: Debug
CMAKEFLAGS: -DCMAKE_BUILD_TYPE=Debug -DLUA_USE_ASSERT=ON -DLUA_USE_APICHECK=ON
@@ -49,6 +49,8 @@ jobs:
- FLAVOR: gdbjit
FLAVORFLAGS: -DLUAJIT_USE_GDBJIT=ON
exclude:
+ - ARCH: ARM64
+ GC64: OFF
# DUALNUM is default for ARM64, no need for additional testing.
- FLAVOR: dualnum
ARCH: ARM64
--
2.30.2
More information about the Tarantool-patches
mailing list