Skip to content

Commit d7666f7

Browse files
authored
Merge pull request #325 from luau-project/fix-ubuntu-ci
fix: LuaJIT on Ubuntu CI
2 parents a61636b + a376832 commit d7666f7

File tree

2 files changed

+18
-22
lines changed

2 files changed

+18
-22
lines changed

.github/workflows/ci.yml

Lines changed: 17 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ jobs:
1313

1414
strategy:
1515
matrix:
16-
version: [5.1.5, 5.2.4, 5.3.6, 5.4.3]
16+
version: [5.1.5, 5.2.4, 5.3.6, 5.4.7]
1717

1818
steps:
1919
- name: Build PUC-Rio Lua ${{ matrix.version }}
@@ -30,7 +30,7 @@ jobs:
3030
run: |
3131
cd "lua-${LUA_VERSION}"
3232
sudo make install
33-
- uses: actions/checkout@v2
33+
- uses: actions/checkout@v4
3434
- name: Install build dependencies
3535
run: source .github/install_deps.bash
3636
- name: Build lgi and the test suite
@@ -59,30 +59,27 @@ jobs:
5959
name: Test on LuaJIT
6060
runs-on: ubuntu-latest
6161

62-
strategy:
63-
matrix:
64-
version: [2.0.5]
65-
6662
steps:
67-
- name: Build LuaJIT ${{ matrix.version }}
68-
env:
69-
LUAJIT_VERSION: ${{ matrix.version }}
70-
LSAN_OPTIONS: exitcode=0
71-
run: |
72-
wget "https://luajit.org/download/LuaJIT-${LUAJIT_VERSION}.tar.gz"
73-
tar xzvf "LuaJIT-${LUAJIT_VERSION}.tar.gz"
74-
cd "LuaJIT-${LUAJIT_VERSION}"
75-
make CC='gcc -g'
63+
- name: Set environment variable to hold LuaJIT checkout directory
64+
run: echo "LUAJIT_CHECKOUT=$(uuidgen)" >> "${{ github.env }}"
65+
- name: Checkout LuaJIT from github mirror
66+
uses: actions/checkout@v4
67+
with:
68+
repository: LuaJIT/LuaJIT
69+
path: "${{ env.LUAJIT_CHECKOUT }}"
70+
- name: Build LuaJIT
71+
run: make -C "${{ env.LUAJIT_CHECKOUT }}" CC='gcc -g'
7672
- name: Install LuaJIT ${{ matrix.version }}
77-
env:
78-
LUAJIT_VERSION: ${{ matrix.version }}
7973
run: |
80-
cd "LuaJIT-${LUAJIT_VERSION}"
81-
sudo make install
74+
sudo make -C "${{ env.LUAJIT_CHECKOUT }}" install
8275
sudo ln -frs /usr/local/bin/luajit-* /usr/local/bin/lua
8376
sudo ln -frs /usr/local/lib/libluajit-*.a /usr/local/lib/liblua.a
8477
sudo ln -frs /usr/local/include/luajit-*/* /usr/local/include/
85-
- uses: actions/checkout@v2
78+
- name: Set LUA_VERSION environment variable from LuaJIT's pkg-config file
79+
run: |
80+
lua_version=$(PKG_CONFIG_PATH="/usr/local/lib/pkgconfig" pkg-config --variable=abiver luajit);
81+
echo "LUA_VERSION=${lua_version}" >> "${{ github.env }}"
82+
- uses: actions/checkout@v4
8683
- name: Install build dependencies
8784
run: source .github/install_deps.bash
8885
- name: Build lgi and the test suite
@@ -100,7 +97,6 @@ jobs:
10097
env:
10198
LSAN_OPTIONS: suppressions=${{ github.workspace }}/.github/lsan.supp
10299
UBSAN_OPTIONS: print_stacktrace=1:report_error_type=1:halt_on_error=1:suppressions=${{ github.workspace }}/.github/ubsan.supp
103-
LUA_VERSION: ${{ matrix.version }}
104100
run: |
105101
source .github/find_asan_ubsan.bash "lgi/corelgilua51.so"
106102
source .github/test_make_install.bash

.github/workflows/msvc-ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ jobs:
2626

2727
env:
2828
WINGTK_URL: https://github.com/wingtk/gvsbuild/releases/download/2024.10.0/GTK${{ matrix.gtk-major-version }}_Gvsbuild_2024.10.0_x64.zip
29-
LUAINSTALLER_URL: https://github.com/luau-project/LuaInstaller/releases/download/v0.2.0.0/LuaInstaller.Console-v0.2.0.0-x64.zip
29+
LUAINSTALLER_URL: https://github.com/luau-project/LuaInstaller/releases/download/v0.3.0.0/LuaInstaller.Console-v0.3.0.0-x64.zip
3030

3131
steps:
3232

0 commit comments

Comments
 (0)