Skip to content

ivykis: add new package#27717

Merged
BKPepe merged 1 commit intoopenwrt:masterfrom
BKPepe:add-ivykis-library-for-syslogng
Oct 28, 2025
Merged

ivykis: add new package#27717
BKPepe merged 1 commit intoopenwrt:masterfrom
BKPepe:add-ivykis-library-for-syslogng

Conversation

@BKPepe
Copy link
Copy Markdown
Member

@BKPepe BKPepe commented Oct 28, 2025

📦 Package Details

Maintainer: @BKPepe

Description:
Libivykis, an asynchronous I/O readiness notification library. This library is required by syslog-ng and potentially other packages that need event-driven I/O handling.

It is required for #27590
cc: @neheb


🧪 Run Testing Details

  • OpenWrt Version: OpenWrt 24.10
  • OpenWrt Target/Subtarget: mvebu
  • OpenWrt Device: Turris Omnia

✅ Formalities

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

@BKPepe BKPepe requested a review from Copilot October 28, 2025 09:59
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a new package for libivykis, an asynchronous I/O readiness notification library required by syslog-ng. The package includes standard OpenWrt build configurations with support for both shared and static libraries.

Key Changes:

  • Adds ivykis version 0.43.2 as a new library package
  • Configures build system with autoreconf, shared/static libraries, and PIC support
  • Implements standard OpenWrt installation targets for development headers and runtime libraries

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@BKPepe BKPepe mentioned this pull request Oct 28, 2025
4 tasks
@BKPepe BKPepe force-pushed the add-ivykis-library-for-syslogng branch from dea96a9 to 6081718 Compare October 28, 2025 10:17
Comment thread libs/ivykis/Makefile Outdated
Comment on lines +39 to +45
--with-pic

ifdef CONFIG_KERNEL_IO_URING
CONFIGURE_ARGS += --enable-io-uring=yes
else
CONFIGURE_ARGS += --enable-io-uring=no
endif
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

can this be simplified with

Suggested change
--with-pic
ifdef CONFIG_KERNEL_IO_URING
CONFIGURE_ARGS += --enable-io-uring=yes
else
CONFIGURE_ARGS += --enable-io-uring=no
endif
--with-pic \
--enable-io-uring=$(if $(CONFIG_KERNEL_IO_URING),yes,no)

Copy link
Copy Markdown
Member Author

@BKPepe BKPepe Oct 28, 2025

Choose a reason for hiding this comment

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

Ha, much better! Thanks! 😊

Libivykis, an asynchronous I/O readiness notification library.
This library is required by syslog-ng and potentially other packages
that need event-driven I/O handling.

Signed-off-by: Josef Schlehofer <pepe.schlehofer@gmail.com>
@BKPepe BKPepe merged commit c92d120 into openwrt:master Oct 28, 2025
12 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.

4 participants