From patchwork Fri Aug 14 07:17:00 2026 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: James Lee X-Patchwork-Id: 95238 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from aws-us-west-2-korg-lkml-1.web.codeaurora.org (localhost.localdomain [127.0.0.1]) by smtp.lore.kernel.org (Postfix) with ESMTP id 9B497C5CFCF for ; Fri, 14 Aug 2026 07:17:18 +0000 (UTC) Received: from codeconstruct.com.au (codeconstruct.com.au [203.29.241.158]) by mx.groups.io with SMTP id smtpd.msgproc02-g2.8004.1786691827797719714 for ; Fri, 14 Aug 2026 00:17:08 -0700 Authentication-Results: mx.groups.io; dkim=fail reason="dkim: body hash did not verify" header.i=@codeconstruct.com.au header.s=2022a header.b=fHkaqglD; spf=pass (domain: codeconstruct.com.au, ip: 203.29.241.158, mailfrom: james@codeconstruct.com.au) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=codeconstruct.com.au; s=2022a; t=1786691825; bh=aFBEx/9KR7QjhxgUH5WVCyUI8Wi5awyfBDtXoKirz1U=; h=From:To:Cc:Subject:Date; b=fHkaqglD7SNGOPYUZmpf923nARAKJ/yimX+T4qgrpkJxrYWhpfS1wkvlatj4gLbap cXhL50a7qlG4YKx2NExOPF6uO9A/LkeOc2hrBSAEHBY/1Xn9HyylNpbLkFGTJUj7u8 zXp3jZW9+dqzKgvcmuHlPYyONXiXZXzYi+HwSybfLszhLddwB95CvAcIvZRzCcdyJl qAhdl/1y35yevR3DrnLUY7+cKVTLoJKpow06uwFjEaOFOA5EwtFzDDRHgV71/IyiOt lQiflj5CVnl0IKUu3uGsPjIJvNncUwWWC1WauANO09t1SWo6ulyYSeJCti8DOvppS4 gbJk+OTiRiynw== Received: by codeconstruct.com.au (Postfix, from userid 10004) id 6C904679E2; Fri, 14 Aug 2026 15:17:05 +0800 (AWST) From: James Lee To: openembedded-devel@lists.openembedded.org Cc: andrew@codeconstruct.com.au, jk@codeconstruct.com.au Subject: [oe] [meta-networking] [PATCH v3 1/2] mctp: split mctp tools and mctpd into separate recipes Date: Fri, 14 Aug 2026 15:17:00 +0800 Message-ID: <20260814071701.3733870-1-james@codeconstruct.com.au> X-Mailer: git-send-email 2.47.3 MIME-Version: 1.0 List-Id: X-Webhook-Received: from 45-33-107-173.ip.linodeusercontent.com [45.33.107.173] by aws-us-west-2-korg-lkml-1.web.codeaurora.org with HTTPS for ; Fri, 14 Aug 2026 07:17:18 -0000 X-Groupsio-URL: https://lists.openembedded.org/g/openembedded-devel/message/129041 The current mctp recipe relies on the upstream's conditional build and install of the mctpd binary if systemd is present, only building the mctp utility otherwise. Future upstream changes allowing running as non-root require more control over the build process. We will need to add a new system user when mctpd is built. Adding a user requires inheriting useradd, which assumes at least one user will be created, failing the recipe otherwise. Building without mctpd would then cause the recipe to fail as conditionally inheriting useradd seems infeasible. To make useradd available for the new upstream release, split the recipe into "mctp" for the utilities and "mctpd" for the daemon, isolating the systemd dependency to "mctpd". This makes the two parts of the source explicitly selectable from user choice, rather than auto-detecting based on the presence of systemd. This will allow the mctpd daemon to be installed without the mctp utilities and allow the utilities to be installed without the daemon in distros without systemd. It will also let us express the useradd rules only when they're needed for the mctpd installation in the upcoming version bump. The mctp/mctpd split cannot be done with different packages within the same recipe. It would require the systemd and useradd classes be added to inherit_defer only when mctpd is to be built, and the library functions cannot be used to detect which packages are built. Signed-off-by: James Lee james@codeconstruct.com.au --- meta-networking/recipes-support/mctp/mctp.inc | 11 +++++ .../recipes-support/mctp/mctp_2.5.bb | 46 ++----------------- .../recipes-support/mctp/mctpd_2.5.bb | 29 ++++++++++++ 3 files changed, 43 insertions(+), 43 deletions(-) create mode 100644 meta-networking/recipes-support/mctp/mctp.inc create mode 100644 meta-networking/recipes-support/mctp/mctpd_2.5.bb diff --git a/meta-networking/recipes-support/mctp/mctp.inc b/meta-networking/recipes-support/mctp/mctp.inc new file mode 100644 index 0000000000..98225cffea --- /dev/null +++ b/meta-networking/recipes-support/mctp/mctp.inc @@ -0,0 +1,11 @@ +HOMEPAGE = "http://www.github.com/CodeConstruct/mctp" +SECTION = "net" +LICENSE = "GPL-2.0-only" +LIC_FILES_CHKSUM = "file://LICENSE;md5=4cc91856b08b094b4f406a29dc61db21" + +SRC_URI = "git://github.com/CodeConstruct/mctp;branch=main;protocol=https;tag=v${PV}" +SRCREV = "07c7a5d526f686da89bc9ae9cad917e46652b272" + +inherit meson pkgconfig + +EXTRA_OEMESON = "-Dtests=false" diff --git a/meta-networking/recipes-support/mctp/mctp_2.5.bb b/meta-networking/recipes-support/mctp/mctp_2.5.bb index a8787d3f2b..5811495c66 100644 --- a/meta-networking/recipes-support/mctp/mctp_2.5.bb +++ b/meta-networking/recipes-support/mctp/mctp_2.5.bb @@ -1,45 +1,5 @@ -SUMMARY = "Management Component Control Protocol utilities" -HOMEPAGE = "http://www.github.com/CodeConstruct/mctp" -SECTION = "net" -LICENSE = "GPL-2.0-only" +require mctp.inc -LIC_FILES_CHKSUM = "file://LICENSE;md5=4cc91856b08b094b4f406a29dc61db21" +SUMMARY = "Management Component Transport Protocol (MCTP) utilities" -SRCREV = "07c7a5d526f686da89bc9ae9cad917e46652b272" - -SRC_URI = "git://github.com/CodeConstruct/mctp;branch=main;protocol=https;tag=v${PV}" - - -inherit meson pkgconfig systemd - -EXTRA_OEMESON = " \ - -Dtests=false \ -" - -PACKAGECONFIG ??= " \ - ${@bb.utils.filter('DISTRO_FEATURES', 'systemd', d)} \ -" - -# mctpd will only be built if pkg-config detects libsystemd; in which case -# we'll want to declare the dep and install the service. -PACKAGECONFIG[systemd] = ",,systemd,libsystemd" -SYSTEMD_SERVICE:${PN} = "mctpd.service mctp.target mctp-local.target" -SYSTEMD_AUTO_ENABLE:${PN} = "enable" - -do_install:append () { - if ${@bb.utils.contains('PACKAGECONFIG', 'systemd', 'true', 'false', d)}; then - install -d ${D}${systemd_system_unitdir} - install -m 0644 ${S}/conf/mctpd.service \ - ${D}${systemd_system_unitdir}/mctpd.service - install -m 0644 ${S}/conf/*.target \ - ${D}${systemd_system_unitdir}/ - install -d ${D}${datadir}/dbus-1/system.d - install -m 0644 ${S}/conf/mctpd-dbus.conf \ - ${D}${datadir}/dbus-1/system.d/mctpd.conf - fi -} - -FILES:${PN} += " \ - ${@bb.utils.contains('DISTRO_FEATURES', 'systemd', \ - '${datadir}/dbus-1/system.d/mctpd.conf', '', d)} \ -" +MESON_TARGET = "mctp mctp-client" diff --git a/meta-networking/recipes-support/mctp/mctpd_2.5.bb b/meta-networking/recipes-support/mctp/mctpd_2.5.bb new file mode 100644 index 0000000000..a0f2f778a3 --- /dev/null +++ b/meta-networking/recipes-support/mctp/mctpd_2.5.bb @@ -0,0 +1,29 @@ +require mctp.inc + +SUMMARY = "Management Component Transport Protocol (MCTP) control protocol daemon" + +inherit systemd features_check + +do_install:append () { + install -d ${D}${systemd_system_unitdir} + install -m 0644 ${S}/conf/mctpd.service \ + ${D}${systemd_system_unitdir}/mctpd.service + install -m 0644 ${S}/conf/*.target \ + ${D}${systemd_system_unitdir}/ + install -d ${D}${datadir}/dbus-1/system.d + install -m 0644 ${S}/conf/mctpd-dbus.conf \ + ${D}${datadir}/dbus-1/system.d/mctpd.conf +} + +SYSTEMD_SERVICE:${PN} = "mctpd.service mctp.target mctp-local.target" + +DEPENDS += "systemd" +REQUIRED_DISTRO_FEATURES = "systemd" + +# Meson adds mctp utility binaries to FILES and do_install(), which cannot be +# overriden by MESON_TARGET. Override FILES and remove binaries to correct. +FILES:${PN} = "${datadir}/dbus-1/system.d/mctpd.conf ${sbindir}/mctpd" + +do_install:append () { + rm -r ${D}${bindir} +}