All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Zk47T" <zizuzacker@gmail.com>
To: openembedded-core@lists.openembedded.org
Cc: Ross.Burton@arm.com
Subject: [PATCH v2 0/3] useradd.bbclass: fix parse error when only USERADD_DEPENDS is set
Date: Mon, 20 Apr 2026 23:39:37 +0700	[thread overview]
Message-ID: <20260420163940.182918-1-zizuzacker@gmail.com> (raw)

When a recipe inherits useradd and only sets USERADD_DEPENDS (to depend
on users/groups created by another recipe) without creating any
users/groups itself, the parse-time sanity check in
update_useradd_after_parse() unconditionally throws a fatal error about
missing USERADD_PACKAGES.

This series fixes the issue by skipping the USERADD_PACKAGES validation
when USERADD_DEPENDS is set, adds a selftest recipe to exercise this
code path, and adds an oe-selftest case to verify the build succeeds.

Changes in v2:
- Added oe-selftest test case in usergrouptests.py (patch 3/3)
  as suggested by Ross Burton

Fixes [YOCTO #15863]

Nguyen Minh Tien (3):
  useradd.bbclass: allow inheriting with only USERADD_DEPENDS set
  meta-selftest: add usegroup-deponly recipe to test USERADD_DEPENDS
    only
  oe-selftest: add test for useradd with only USERADD_DEPENDS

 .../selftest-users/usegroup-deponly.bb        | 25 +++++++++++++++++++
 meta/classes/useradd.bbclass                  |  5 ++++
 .../lib/oeqa/selftest/cases/usergrouptests.py |  6 +++++
 3 files changed, 36 insertions(+)
 create mode 100644 meta-selftest/recipes-test/selftest-users/usegroup-deponly.bb

-- 
2.34.1



             reply	other threads:[~2026-04-20 16:40 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-04-20 16:39 Zk47T [this message]
2026-04-20 16:39 ` [PATCH v2 1/3] useradd.bbclass: allow inheriting with only USERADD_DEPENDS set Zk47T
2026-04-20 16:39 ` [PATCH v2 2/3] meta-selftest: add usegroup-deponly recipe to test USERADD_DEPENDS only Zk47T
2026-04-20 16:39 ` [PATCH v2 3/3] oe-selftest: add test for useradd with only USERADD_DEPENDS Zk47T
2026-04-23  6:44   ` [OE-core] " Mathieu Dubois-Briand
2026-04-23 10:04     ` [PATCH v3 0/3] useradd.bbclass: fix parse error when only USERADD_DEPENDS is set Zk47T
2026-04-23 10:04       ` [PATCH v3 1/3] useradd.bbclass: allow inheriting with only USERADD_DEPENDS set Zk47T
2026-04-23 10:04       ` [PATCH v3 2/3] meta-selftest: add usegroup-deponly recipe to test USERADD_DEPENDS only Zk47T
2026-04-23 10:04       ` [PATCH v3 3/3] oe-selftest: add test for useradd with only USERADD_DEPENDS Zk47T
2026-04-23 10:09         ` Zk47T

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=20260420163940.182918-1-zizuzacker@gmail.com \
    --to=zizuzacker@gmail.com \
    --cc=Ross.Burton@arm.com \
    --cc=openembedded-core@lists.openembedded.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.