public inbox for openembedded-core@lists.openembedded.org
 help / color / mirror / Atom feed
From: ankur.tyagi85@gmail.com
To: openembedded-core@lists.openembedded.org
Cc: Gyorgy Sarvari <skandigraun@gmail.com>,
	Mathieu Dubois-Briand <mathieu.dubois-briand@bootlin.com>,
	Richard Purdie <richard.purdie@linuxfoundation.org>,
	Ankur Tyagi <ankur.tyagi85@gmail.com>
Subject: [OE-core][whinlatter][PATCH 08/12] seatd: fix packaging error with systemd DISTRO_FEATURES
Date: Wed, 25 Mar 2026 15:18:52 +1300	[thread overview]
Message-ID: <20260325021856.4053666-8-ankur.tyagi85@gmail.com> (raw)
In-Reply-To: <20260325021856.4053666-1-ankur.tyagi85@gmail.com>

From: Gyorgy Sarvari <skandigraun@gmail.com>

The systemd unit file for this recipe is only installed in case
the target system's init system is systemd. It is achieved by
inspecting the VIRTUAL-RUNTIME_init_manager variable.

However the systemd class was inherited unconditionally. This caused
a failure in do_package task, in case systemd is present in the
DISTRO_FEATURES but the system's init manager is sysvinit: in this case
the systemd unit file is not installed, however systemd.bbclass is
still trying to register is as a startup service. At this point
it failed:

ERROR: seatd-0.9.1-r0 do_package: Didn't find service unit 'seatd.service', specified in SYSTEMD_SERVICE:seatd.

To avoid this, install the systemd unit file unconditionally, regardless
of the used init system.

Signed-off-by: Gyorgy Sarvari <skandigraun@gmail.com>
Signed-off-by: Mathieu Dubois-Briand <mathieu.dubois-briand@bootlin.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit c7f157ad1207567ef3614ee4f6e755bccf60a3f4)
Signed-off-by: Ankur Tyagi <ankur.tyagi85@gmail.com>
---
 meta/recipes-core/seatd/seatd_0.9.1.bb | 2 --
 1 file changed, 2 deletions(-)

diff --git a/meta/recipes-core/seatd/seatd_0.9.1.bb b/meta/recipes-core/seatd/seatd_0.9.1.bb
index f9f72c9e82..3be27dda9d 100644
--- a/meta/recipes-core/seatd/seatd_0.9.1.bb
+++ b/meta/recipes-core/seatd/seatd_0.9.1.bb
@@ -34,8 +34,6 @@ PACKAGECONFIG[systemd] = "-Dlibseat-logind=systemd,,systemd"
 do_install:append() {
         if [ "${VIRTUAL-RUNTIME_init_manager}" != "systemd" ]; then
                 install -Dm755 ${UNPACKDIR}/init ${D}/${sysconfdir}/init.d/seatd
-        else
-                install -Dm644 ${S}/contrib/systemd/seatd.service ${D}${systemd_unitdir}/system/seatd.service
         fi
         install -Dm644 ${S}/contrib/systemd/seatd.service ${D}${systemd_unitdir}/system/seatd.service
 }


  parent reply	other threads:[~2026-03-25  2:20 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-03-25  2:18 [OE-core][whinlatter][PATCH 01/12] ccache: upgrade 4.12.2 -> 4.12.3 ankur.tyagi85
2026-03-25  2:18 ` [OE-core][whinlatter][PATCH 02/12] libsoup: upgrade 3.6.5 -> 3.6.6 ankur.tyagi85
2026-03-25  2:18 ` [OE-core][whinlatter][PATCH 03/12] libsoup: fix CVE-2025-32049/CVE-2026-1539 ankur.tyagi85
2026-03-25  2:18 ` [OE-core][whinlatter][PATCH 04/12] libxmlb: upgrade 0.3.24 -> 0.3.25 ankur.tyagi85
2026-03-25  2:18 ` [OE-core][whinlatter][PATCH 05/12] ca-certificates: upgrade 20250419 -> 20260223 ankur.tyagi85
2026-03-25  2:18 ` [OE-core][whinlatter][PATCH 06/12] tzdata,tzcode-native: Upgrade 2025b -> 2025c ankur.tyagi85
2026-03-25  2:18 ` [OE-core][whinlatter][PATCH 07/12] seatd: Create seat user and package systemd service ankur.tyagi85
2026-04-01 14:58   ` Yoann Congal
2026-04-05  2:18     ` Ankur Tyagi
2026-03-25  2:18 ` ankur.tyagi85 [this message]
2026-03-25  2:18 ` [OE-core][whinlatter][PATCH 09/12] seatd: upgrade 0.9.1 -> 0.9.2 ankur.tyagi85
2026-03-25  2:18 ` [OE-core][whinlatter][PATCH 10/12] seatd: upgrade 0.9.2 -> 0.9.3 ankur.tyagi85
2026-03-25  2:18 ` [OE-core][whinlatter][PATCH 11/12] e2fsprogs: upgrade 1.47.3 -> 1.47.4 ankur.tyagi85
2026-04-01 15:04   ` Yoann Congal
2026-04-02  0:06     ` Ankur Tyagi
2026-03-25  2:18 ` [OE-core][whinlatter][PATCH 12/12] libinput: upgrade 1.29.1 -> 1.29.2 ankur.tyagi85
2026-04-01 15:21   ` Yoann Congal
2026-04-02  0:01     ` Ankur Tyagi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20260325021856.4053666-8-ankur.tyagi85@gmail.com \
    --to=ankur.tyagi85@gmail.com \
    --cc=mathieu.dubois-briand@bootlin.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=richard.purdie@linuxfoundation.org \
    --cc=skandigraun@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox