Skip to content

luajit2: update to 2026-07-31 and refresh RISC-V 64 patch - #30282

Merged
BKPepe merged 1 commit into
openwrt:masterfrom
BKPepe:luajit2-update-2026-07-31
Aug 15, 2026
Merged

luajit2: update to 2026-07-31 and refresh RISC-V 64 patch#30282
BKPepe merged 1 commit into
openwrt:masterfrom
BKPepe:luajit2-update-2026-07-31

Conversation

@BKPepe

@BKPepe BKPepe commented Aug 15, 2026

Copy link
Copy Markdown
Member

📦 Package Details

Maintainer: @jmarcet

Description:

Updates luajit2 from 2026-02-27 to 2026-07-31 (openresty/luajit2 52f52587) and refreshes the RISC-V 64 patch from the current state of openresty/luajit2#236.

The riscv64 build has been failing since openresty/luajit2 commit 68354f44 added a TraceNo argument to asm_tail_prep(). The snapshot in 020-riscv64-support.patch still had the one-argument form, so the build stopped with:

lj_asm.c:2559:5: error: too many arguments to function 'asm_tail_prep'

Upstream PR #236 has since been adapted, and the refreshed patch picks that up.

PKG_MIRROR_HASH was computed with make package/luajit2/check FIXUP=1 in the official ghcr.io/openwrt/sdk:x86_64-master image. All five patches apply cleanly against the new version and only 020 needed refreshing.


🧪 Run Testing Details

  • OpenWrt Version: master
  • OpenWrt Target/Subtarget: mpc85xx / p2020 (powerpc_8548)
  • OpenWrt Device: Turris 1.0 / 1.1 (e500v2, big-endian, musl, soft-float)

The powerpc side of this package was run-tested on the device for the previous change (#30280). This PR is a version bump plus a patch refresh, so the CI build is the primary check here, in particular riscv64_generic and powerpc_8548.


✅ Formalities

  • I have reviewed the CONTRIBUTING.md file for detailed contributing guidelines.

If your PR contains a patch:

  • It can be applied using git am
  • It has been refreshed to avoid offsets, fuzzes, etc., using
    make package/<your-package>/refresh V=s
  • It is structured in a way that it is potentially upstreamable

Also refresh 020-riscv64-support.patch from the current state of
openresty/luajit2#236. The previous snapshot defined
asm_tail_prep(ASMState *as), but since openresty/luajit2 commit
68354f44 the callers pass a second TraceNo argument, so the riscv64
build failed with:

  lj_asm.c:2559:5: error: too many arguments to function 'asm_tail_prep'

The refreshed patch carries the upstream adaptation and restores the
build. The other four patches apply unchanged.

Signed-off-by: Josef Schlehofer <pepe.schlehofer@gmail.com>
Copilot AI lite review requested due to automatic review settings August 15, 2026 10:13

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@BKPepe

BKPepe commented Aug 15, 2026

Copy link
Copy Markdown
Member Author

CI/CD:

luajit2: Use generic tests
WARNING: opening /ci/packages.adb: UNTRUSTED signature
luajit2: [pass] File /usr/bin/luajit2 is executable
luajit2: [skip] Version check (/usr/bin/luajit2)
luajit2: [pass] Binary /usr/bin/luajit2 does not contain any hardcoded build paths
luajit2: [pass] Binary /usr/bin/luajit2 is stripped
luajit2: [pass] All linked libraries for /usr/bin/luajit2 are present
luajit2: [pass] Symlink /usr/lib/libluajit-5.1.so points to an existing file
luajit2: [pass] Symlink /usr/lib/libluajit-5.1.so.2 points to an existing file
luajit2: [pass] Binary /usr/lib/libluajit-5.1.so.2.1.1785460716 does not contain any hardcoded build paths
luajit2: [pass] Binary /usr/lib/libluajit-5.1.so.2.1.1785460716 is stripped
luajit2: [pass] All linked libraries for /usr/lib/libluajit-5.1.so.2.1.1785460716 are present
luajit2: [pass] Library /usr/lib/libluajit-5.1.so.2.1.1785460716 has SONAME 'libluajit-5.1.so.2'
luajit2: [pass] SONAME link for /usr/lib/libluajit-5.1.so.2.1.1785460716 is correct
LuaJIT 2.1.1785460716 -- Copyright (C) 2005-2026 Mike Pall. https://luajit.org/
luajit2: [pass] Version check override
luajit2: Generic tests passed

Merging! :)

@BKPepe
BKPepe merged commit 578758e into openwrt:master Aug 15, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants