From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: openembedded-core@lists.openembedded.org
Subject: [PATCH 3/3] shadow: Drop pointless patch
Date: Tue, 21 Apr 2026 11:41:47 +0100 [thread overview]
Message-ID: <20260421104147.1994119-3-richard.purdie@linuxfoundation.org> (raw)
In-Reply-To: <20260421104147.1994119-1-richard.purdie@linuxfoundation.org>
As far as I can tell, this patch is a no-op and doens't change the code.
As such, I think it just complicates things and can be removed.
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
...nexpected-open-failure-in-chroot-env.patch | 48 -------------------
meta/recipes-extended/shadow/shadow_4.19.4.bb | 1 -
2 files changed, 49 deletions(-)
delete mode 100644 meta/recipes-extended/shadow/files/commonio.c-fix-unexpected-open-failure-in-chroot-env.patch
diff --git a/meta/recipes-extended/shadow/files/commonio.c-fix-unexpected-open-failure-in-chroot-env.patch b/meta/recipes-extended/shadow/files/commonio.c-fix-unexpected-open-failure-in-chroot-env.patch
deleted file mode 100644
index 699269ed643..00000000000
--- a/meta/recipes-extended/shadow/files/commonio.c-fix-unexpected-open-failure-in-chroot-env.patch
+++ /dev/null
@@ -1,48 +0,0 @@
-From f7b765c022e4cad9140ac44712885c66e149abdc Mon Sep 17 00:00:00 2001
-From: Chen Qi <Qi.Chen@windriver.com>
-Date: Thu, 17 Jul 2014 15:53:34 +0800
-Subject: [PATCH] commonio.c-fix-unexpected-open-failure-in-chroot-env
-
-Upstream-Status: Inappropriate [OE specific]
-
-commonio.c: fix unexpected open failure in chroot environment
-
-When using commands with '-R <newroot>' option in our pseudo environment,
-we would usually get the 'Pemission Denied' error. This patch serves as
-a workaround to this problem.
-
-Note that this patch doesn't change the logic in the code, it just expands
-the codes.
-
-Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
----
- lib/commonio.c | 16 ++++++++++++----
- 1 file changed, 12 insertions(+), 4 deletions(-)
-
-diff --git a/lib/commonio.c b/lib/commonio.c
-index 4d83e83..9ee0e13 100644
---- a/lib/commonio.c
-+++ b/lib/commonio.c
-@@ -604,10 +604,18 @@ int commonio_open (struct commonio_db *db, int mode)
- db->cursor = NULL;
- db->changed = false;
-
-- fd = open (db->filename,
-- (db->readonly ? O_RDONLY : O_RDWR)
-- | O_NOCTTY | O_NONBLOCK | O_NOFOLLOW | O_CLOEXEC);
-- saved_errno = errno;
-+ if (db->readonly) {
-+ fd = open (db->filename,
-+ (true ? O_RDONLY : O_RDWR)
-+ | O_NOCTTY | O_NONBLOCK | O_NOFOLLOW | O_CLOEXEC);
-+ saved_errno = errno;
-+ } else {
-+ fd = open (db->filename,
-+ (false ? O_RDONLY : O_RDWR)
-+ | O_NOCTTY | O_NONBLOCK | O_NOFOLLOW| O_CLOEXEC);
-+ saved_errno = errno;
-+ }
-+
- db->fp = NULL;
- if (fd >= 0) {
- #ifdef WITH_TCB
diff --git a/meta/recipes-extended/shadow/shadow_4.19.4.bb b/meta/recipes-extended/shadow/shadow_4.19.4.bb
index 3ab9ae9c287..94f155641cc 100644
--- a/meta/recipes-extended/shadow/shadow_4.19.4.bb
+++ b/meta/recipes-extended/shadow/shadow_4.19.4.bb
@@ -23,7 +23,6 @@ SRC_URI:append:class-target = " \
"
SRC_URI:append:class-native = " \
- file://commonio.c-fix-unexpected-open-failure-in-chroot-env.patch \
file://disable_syslog.patch \
file://notallylog.patch \
"
next prev parent reply other threads:[~2026-04-21 10:41 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-04-21 10:41 [PATCH 1/3] pseudo: Update to 1.7.4 Richard Purdie
2026-04-21 10:41 ` [PATCH 2/3] shadow: Disable tallylog execution Richard Purdie
2026-04-21 11:00 ` Patchtest results for " patchtest
2026-04-21 10:41 ` Richard Purdie [this message]
2026-04-22 5:51 ` [OE-core] [PATCH 3/3] shadow: Drop pointless patch ChenQi
2026-04-22 6:44 ` Richard Purdie
2026-04-21 11:41 ` [OE-core] [PATCH 1/3] pseudo: Update to 1.7.4 Yoann Congal
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=20260421104147.1994119-3-richard.purdie@linuxfoundation.org \
--to=richard.purdie@linuxfoundation.org \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox