Skip to content

Conversation

@graysky2
Copy link
Contributor

@graysky2 graysky2 commented Dec 25, 2025

📦 Package Details

Maintainer: @jow-
(You can find this by checking the history of the package Makefile.)

Description:

This upstream release allows clean builds against linux 6.18. Fix LUA
PacketScript build on 6.18 by switching to ccflags-y and restoring the
required -isystem include path. Remove compat_xtables since the module
no longer exists in 3.30 and drop all dependent kmod references.

Note that upstream tarball changed from xz to zst.


🧪 Run Testing Details

  • OpenWrt Version: SNAPSHOT
  • OpenWrt Target/Subtarget:
  • OpenWrt Device:

✅ 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
    (e.g., subject line, commit description, etc.)
    We must try to upstream patches to reduce maintenance burden.

@LGA1150
Copy link
Contributor

LGA1150 commented Dec 25, 2025

v3.30 fixed that

https://codeberg.org/jengelh/xtables-addons/commit/438a7e519ac4f71d8423e323bc2d1aabfb059692

@graysky2 graysky2 changed the title xtables-addons: disable for linux 6.18 xtables-addons: update to 3.30 Dec 25, 2025
@graysky2
Copy link
Contributor Author

thanks @LGA1150

This upstream release allows clean builds against linux 6.18. Fix LUA
PacketScript build on 6.18 by switching to ccflags-y and restoring the
required -isystem include path. Remove compat_xtables since the module
no longer exists in 3.30 and drop all dependent kmod references.

Note that upstream tarball changed from xz to zst.

Signed-off-by: John Audia <therealgraysky@proton.me>
@LGA1150
Copy link
Contributor

LGA1150 commented Dec 27, 2025

Package failed to build on 32-bit because of signness mismatch. do_div expects a u64 dividend, but time64_t is s64.
@jengelh

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants