syslog-ng: switch to CMake by BKPepe · Pull Request #27590 · openwrt/packages

BKPepe · 2025-10-02T08:00:59Z

Maintainer: me

Description:
This PR adds:

ivykis as library
syslog-ng switch to CMake instead of autotools and add ivykis as dependency

🧪 Run Testing Details

So far so no, will be done later, compile tested

-------------------------------------------
syslog-ng Open Source Edition 4.10.0 configured
--------------- Environment ---------------
CMAKE_HOST_SYSTEM                   Darwin-25.0.0
CMAKE_HOST_SYSTEM_NAME              Darwin
CMAKE_HOST_SYSTEM_PROCESSOR         arm64
CMAKE_HOST_SYSTEM_VERSION           25.0.0
---------------- Compilers ----------------
CMAKE_C_COMPILER                    powerpc-openwrt-linux-musl-gcc[powerpc-openwrt-linux-musl-gcc (OpenWrt GCC 13.3.0 r28878+90-bb14c607a3) 13.3.0] - /Volumes/OpenWrt/turris1x-hbk/build/staging_dir/toolchain-powerpc_8548_gcc-13.3.0_musl/bin/powerpc-openwrt-linux-musl-gcc
CMAKE_CXX_COMPILER                  powerpc-openwrt-linux-musl-g++[powerpc-openwrt-linux-musl-g++ (OpenWrt GCC 13.3.0 r28878+90-bb14c607a3) 13.3.0] - /Volumes/OpenWrt/turris1x-hbk/build/staging_dir/toolchain-powerpc_8548_gcc-13.3.0_musl/bin/powerpc-openwrt-linux-musl-g++
--------------- Compilation ---------------
CMAKE_BUILD_TYPE                    Release
ENABLE_CPP                          Off
ENABLE_EXTRA_WARNINGS               On
ENABLE_FORCE_GNU99                  Off
ENV_LD_LIBRARY_PATH                 Off
----------------- Testing -----------------
BUILD_TESTING                       Off
---------------- Man pages ----------------
ENABLE_MANPAGES                     Off
ENABLE_MANPAGES_INSTALL             Auto (Off)
WITH_MANPAGES                       local
---------------- Features -----------------
ENABLE_ENV_WRAPPER                  Off
ENABLE_FORCED_SERVER_MODE           On
ENABLE_GCOV                         Off
ENABLE_GPROF                        Off
ENABLE_IPV6                         On
ENABLE_LINUX_CAPS                   Off
ENABLE_MEMTRACE                     Off
ENABLE_PERF                         Off
ENABLE_SPOOF_SOURCE                 Off
ENABLE_STACKDUMP                    Off
ENABLE_SYSTEMD                      Off
ENABLE_TCP_WRAPPER                  Off
SANITIZER                           Off
--------------- Sub-modules ---------------
IVYKIS_SOURCE                       system
----------------- Modules -----------------
ENABLE_AFAMQP                       Off
ENABLE_AFSMTP                       Off
ENABLE_AFSNMP                       Off
ENABLE_AFUSER                       On
ENABLE_AMQP                         Off
ENABLE_APPMODEL                     On
ENABLE_AZURE_AUTH_HEADER            On
ENABLE_CLOUD_AUTH                   Off
ENABLE_CURL                         On
ENABLE_EBPF                         Off
ENABLE_EXAMPLE_MODULES              Off
ENABLE_GEOIP2                       Off
ENABLE_GETENT                       On
ENABLE_GRPC                         Off
ENABLE_HOOK_COMMANDS                On
ENABLE_HTTP                         Off
ENABLE_JAVA                         Off
ENABLE_JAVA_MODULES                 Off
ENABLE_JOURNALD                     Off
ENABLE_JSON                         On
ENABLE_KAFKA                        Off
ENABLE_MAP_VALUE_PAIRS              On
ENABLE_MONGODB                      Off
ENABLE_MQTT                         Off
ENABLE_NATIVE                       On
ENABLE_OPENBSD_SYS_DRIVER           Off
ENABLE_PACCT                        On
ENABLE_PYTHON                       Off
ENABLE_PYTHON_MODULES               Off
ENABLE_REDIS                        Off
ENABLE_RIEMANN                      Off
ENABLE_SMTP                         Off
ENABLE_SQL                          On
ENABLE_STARDATE                     On
ENABLE_STOMP                        On
ENABLE_SUN_STREAMS                  Off
ENABLE_XML                          On
-------------------------------------------

OpenWrt Version:
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.}

Copilot

Pull Request Overview

Adds new ivykis library package and migrates syslog-ng build system from autotools to CMake while introducing ivykis as a dependency.

Introduces libs/ivykis package (headers, static and shared libs, pkgconfig file)
Replaces syslog-ng autotools configure flags with CMake options
Extends syslog-ng dependencies to include ivykis and adjusts install procedure

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File	Description
libs/ivykis/Makefile	New package definition for ivykis (build, install, dev install)
admin/syslog-ng/Makefile	Switch to CMake, add ivykis dependency, replace configure args with CMake options, adjust install cleanup

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

Copilot

Pull Request Overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

BKPepe · 2025-10-03T15:31:48Z

@neheb Could you take a look at it, please?

neheb · 2025-10-17T22:15:57Z

Package syslog-ng is missing dependencies for the following libraries:
libcap.so.2

BKPepe · 2025-10-20T09:00:14Z

Thanks for testing, @neheb. I do have minimal build on my machine, thus that is why I have not experienced it. I will disable it. :)

neheb · 2025-10-21T19:33:55Z

btw there seems to be some whitespace. My editor did this:

@@ -124,7 +124,7 @@ define Package/syslog-ng/install

        $(INSTALL_DIR) $(1)/usr/lib
        $(CP) $(PKG_INSTALL_DIR)/usr/lib/*.so* $(1)/usr/lib/
-
+
        $(INSTALL_DIR) $(1)/usr/lib/syslog-ng
        $(CP) $(PKG_INSTALL_DIR)/usr/lib/syslog-ng/* $(1)/usr/lib/syslog-ng/

GeorgeSapkin · 2025-10-21T19:52:48Z

Might as well update to 4.10.2.

BKPepe · 2025-10-21T21:15:07Z

Sure - #27684

BKPepe · 2025-10-28T09:43:57Z

At this point, it appears that syslog-ng requires dependencies even though it shouldn't, because the features that require them are disabled. Therefore, to prevent merging, I'm putting this on hold until this issue is resolved.

BKPepe · 2025-10-28T09:59:28Z

Ivykis dependency will be added in separate PR - #27717

BKPepe · 2026-04-27T09:53:26Z

syslog-ng released new version together with syslog-ng/syslog-ng#5553, so I will update syslog-ng to this version and then I will come back to this PR.

BKPepe · 2026-04-28T21:14:13Z

Updated in #29256

GeorgeSapkin · 2026-05-05T20:36:56Z

PKG_RELEASE 👀

Oops! Bumped. Still I need to have working build env first. I do have some issues on MacOS once again. 😆 I am not able to compile OpenWrt toolchain so far. I will need to look at it to be able to continue with this PR.

Reference: openwrt/openwrt#23122

you can deselect Build gdb if you select Advanced configuration options (for developers) ---> Toolchain Options

I know, but proper solution is to fix that. 😇

- Using external ivykis package instead of internal build - Disable optional modules (AMQP, Redis, MongoDB, etc.) Signed-off-by: Josef Schlehofer <pepe.schlehofer@gmail.com>

BKPepe requested a review from Copilot October 2, 2025 08:00

Copilot AI reviewed Oct 2, 2025

View reviewed changes

Comment thread libs/ivykis/Makefile

Comment thread libs/ivykis/Makefile Outdated

Comment thread admin/syslog-ng/Makefile

Comment thread admin/syslog-ng/Makefile Outdated

BKPepe force-pushed the CMake_syslog branch 5 times, most recently from 948d9e0 to 98efa11 Compare October 2, 2025 11:15

BKPepe requested a review from Copilot October 2, 2025 11:27

Copilot AI reviewed Oct 2, 2025

View reviewed changes

Comment thread admin/syslog-ng/Makefile Outdated

Comment thread admin/syslog-ng/Makefile

BKPepe force-pushed the CMake_syslog branch 3 times, most recently from 6d66679 to 604ad67 Compare October 2, 2025 18:30

BKPepe marked this pull request as ready for review October 2, 2025 21:29

neheb reviewed Oct 3, 2025

View reviewed changes

Comment thread admin/syslog-ng/Makefile Outdated

neheb reviewed Oct 3, 2025

View reviewed changes

Comment thread admin/syslog-ng/Makefile Outdated

BKPepe force-pushed the CMake_syslog branch 5 times, most recently from 946afb0 to 1ced1bf Compare October 9, 2025 09:10

BKPepe requested a review from GeorgeSapkin October 9, 2025 09:56

neheb approved these changes Oct 11, 2025

View reviewed changes

neheb reviewed Oct 11, 2025

View reviewed changes

Comment thread libs/ivykis/Makefile

neheb reviewed Oct 11, 2025

View reviewed changes

Comment thread libs/ivykis/Makefile Outdated

BKPepe force-pushed the CMake_syslog branch from 9478e15 to 7d0e002 Compare October 14, 2025 11:36

BKPepe mentioned this pull request Oct 20, 2025

Make libcap library optional if Linux capabilities are disabled (CMake build) syslog-ng/syslog-ng#5544

Closed

GeorgeSapkin added Add package chore labels Oct 21, 2025

GeorgeSapkin added enhancement and removed chore labels Oct 21, 2025

BKPepe added the blocked label Oct 28, 2025

BKPepe mentioned this pull request Oct 28, 2025

ivykis: add new package #27717

Merged

1 task

BKPepe changed the title ~~syslog-ng: switch to CMake + add dependency ivykis for it~~ syslog-ng: switch to CMake Oct 28, 2025

BKPepe force-pushed the CMake_syslog branch from 7d0e002 to b2e2e8e Compare May 5, 2026 20:33

GeorgeSapkin reviewed May 5, 2026

View reviewed changes

BKPepe force-pushed the CMake_syslog branch from b2e2e8e to eaa2172 Compare May 5, 2026 21:10

github-actions Bot removed the Add package label May 5, 2026

syslog-ng: use CMake instead of autotools

e660972

- Using external ivykis package instead of internal build - Disable optional modules (AMQP, Redis, MongoDB, etc.) Signed-off-by: Josef Schlehofer <pepe.schlehofer@gmail.com>

BKPepe force-pushed the CMake_syslog branch from eaa2172 to e660972 Compare May 6, 2026 06:06

BKPepe removed the blocked label May 6, 2026

Conversation

BKPepe commented Oct 2, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🧪 Run Testing Details

✅ Formalities

If your PR contains a patch:

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Uh oh!

Uh oh!

BKPepe commented Oct 3, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

neheb commented Oct 17, 2025

Uh oh!

BKPepe commented Oct 20, 2025

Uh oh!

neheb commented Oct 21, 2025

Uh oh!

GeorgeSapkin commented Oct 21, 2025

Uh oh!

BKPepe commented Oct 21, 2025

Uh oh!

BKPepe commented Oct 28, 2025

Uh oh!

BKPepe commented Oct 28, 2025

Uh oh!

BKPepe commented Apr 27, 2026

Uh oh!

BKPepe commented Apr 28, 2026

Uh oh!

GeorgeSapkin May 5, 2026

Choose a reason for hiding this comment

Uh oh!

BKPepe May 5, 2026

Choose a reason for hiding this comment

Uh oh!

neheb May 6, 2026

Choose a reason for hiding this comment

Uh oh!

BKPepe May 6, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

BKPepe commented Oct 2, 2025 •

edited

Loading