[PATCH 0/1] e2fsprogs: 1.47.1 -> 1.47.3

[PATCH 0/1] e2fsprogs: 1.47.1 -> 1.47.3

[liezhi.yang]

From: Robert Yang <liezhi.yang@windriver.com>

The following changes since commit 134880baa4a971660e0800f70404d15c758fb7db:

  clang: remove obsolete 'terminfo' PACKAGECONFIG (2025-07-17 10:32:41 +0100)

are available in the Git repository at:

  https://github.com/robertlinux/yocto rbt/e2fsprogs
  https://github.com/robertlinux/yocto/tree/rbt/e2fsprogs

Robert Yang (1):
  e2fsprogs: 1.47.1 -> 1.47.3

 ...-libext2fs-fix-std-c23-build-failure.patch | 42 -------------------
 ...2fsprogs_1.47.1.bb => e2fsprogs_1.47.2.bb} |  8 ++--
 2 files changed, 4 insertions(+), 46 deletions(-)
 delete mode 100644 meta/recipes-devtools/e2fsprogs/e2fsprogs/0001-libext2fs-fix-std-c23-build-failure.patch
 rename meta/recipes-devtools/e2fsprogs/{e2fsprogs_1.47.1.bb => e2fsprogs_1.47.2.bb} (96%)

-- 
2.49.0

[PATCH 1/1] e2fsprogs: 1.47.1 -> 1.47.3

[liezhi.yang]

From: Robert Yang <liezhi.yang@windriver.com>

* Remove backported patch 0001-libext2fs-fix-std-c23-build-failure.patch
* Fixed QA Issue:
  ERROR: e2fsprogs-1.47.2-r0 do_package_qa: QA Issue: File /usr/lib/e2fsprogs/ptest/test/test_one in package e2fsprogs-ptest contains reference to TMPDIR [buildpaths]

Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
---
 ...-libext2fs-fix-std-c23-build-failure.patch | 42 -------------------
 ...2fsprogs_1.47.1.bb => e2fsprogs_1.47.2.bb} |  8 ++--
 2 files changed, 4 insertions(+), 46 deletions(-)
 delete mode 100644 meta/recipes-devtools/e2fsprogs/e2fsprogs/0001-libext2fs-fix-std-c23-build-failure.patch
 rename meta/recipes-devtools/e2fsprogs/{e2fsprogs_1.47.1.bb => e2fsprogs_1.47.2.bb} (96%)

diff --git a/meta/recipes-devtools/e2fsprogs/e2fsprogs/0001-libext2fs-fix-std-c23-build-failure.patch b/meta/recipes-devtools/e2fsprogs/e2fsprogs/0001-libext2fs-fix-std-c23-build-failure.patch
deleted file mode 100644
index 01ab9d5afb..0000000000
--- a/meta/recipes-devtools/e2fsprogs/e2fsprogs/0001-libext2fs-fix-std-c23-build-failure.patch
+++ /dev/null
@@ -1,42 +0,0 @@
-From 72dcef02bee9924c4d5b3dc6e7ef4d07becebcc6 Mon Sep 17 00:00:00 2001
-From: Rudi Heitbaum <rudi@heitbaum.com>
-Date: Fri, 22 Nov 2024 12:36:32 +0000
-Subject: [PATCH] libext2fs: fix -std=c23 build failure
-
-gcc-15 switched to -std=c23 by default:
-
-    https://gcc.gnu.org/git/?p=gcc.git;a=commitdiff;h=55e3bd376b2214e200fa76d12b67ff259b06c212
-
-As a result `e2fsprogs` fails the build so only typedef int bool
-for __STDC_VERSION__ <= 201710L (C17)
-
-    ../../../lib/ext2fs/tdb.c:113:13: error: two or more data types in declaration specifiers
-    ../../../lib/ext2fs/tdb.c:113:1: warning: useless type name in empty declaration
-      113 | typedef int bool;
-          | ^~~~~~~
-
-https://github.com/tytso/e2fsprogs/issues/202
-
-Upstream-Status: Backport [https://github.com/tytso/e2fsprogs/commit/49fd04d77b3244c6c6990be41142168eef373aef]
-Signed-off-by: Rudi Heitbaum <rudi@heitbaum.com>
-Link: https://lore.kernel.org/r/Z0B60JhdvT9bpSQ6@6f91903e89da
-Signed-off-by: Theodore Ts'o <tytso@mit.edu>
-Signed-off-by: Khem Raj <raj.khem@gmail.com>
----
- lib/ext2fs/tdb.c | 2 ++
- 1 file changed, 2 insertions(+)
-
-diff --git a/lib/ext2fs/tdb.c b/lib/ext2fs/tdb.c
-index b07b2917..98dc95d8 100644
---- a/lib/ext2fs/tdb.c
-+++ b/lib/ext2fs/tdb.c
-@@ -110,7 +110,9 @@ static char *rep_strdup(const char *s)
- #endif
- #endif
- 
-+#if defined __STDC__ && defined __STDC_VERSION__ && __STDC_VERSION__ <= 201710L
- typedef int bool;
-+#endif
- 
- #include "tdb.h"
- 
diff --git a/meta/recipes-devtools/e2fsprogs/e2fsprogs_1.47.1.bb b/meta/recipes-devtools/e2fsprogs/e2fsprogs_1.47.2.bb
similarity index 96%
rename from meta/recipes-devtools/e2fsprogs/e2fsprogs_1.47.1.bb
rename to meta/recipes-devtools/e2fsprogs/e2fsprogs_1.47.2.bb
index 022b016f12..5a04591af0 100644
--- a/meta/recipes-devtools/e2fsprogs/e2fsprogs_1.47.1.bb
+++ b/meta/recipes-devtools/e2fsprogs/e2fsprogs_1.47.2.bb
@@ -4,14 +4,13 @@ SRC_URI += "file://remove.ldconfig.call.patch \
            file://run-ptest \
            file://ptest.patch \
            file://mkdir_p.patch \
-           file://0001-libext2fs-fix-std-c23-build-failure.patch \
            "
 SRC_URI:append:class-native = " \
            file://e2fsprogs-fix-missing-check-for-permission-denied.patch \
            file://quiet-debugfs.patch \
            "
 
-SRCREV = "b571b9b4240739a982e8bca62cfc914a3b50190a"
+SRCREV = "da631e117dcf8797bfda0f48bdaa05ac0fbcf7af"
 UPSTREAM_CHECK_GITTAGREGEX = "v(?P<pver>\d+\.\d+(\.\d+)*)$"
 
 EXTRA_OECONF += "--libdir=${base_libdir} --sbindir=${base_sbindir} \
@@ -119,7 +118,7 @@ ALTERNATIVE:${PN}-tune2fs = "tune2fs"
 ALTERNATIVE_LINK_NAME[tune2fs] = "${base_sbindir}/tune2fs"
 
 RDEPENDS:e2fsprogs-e2scrub = "bash"
-RDEPENDS:${PN}-ptest += "coreutils procps bash bzip2 diffutils perl sed"
+RDEPENDS:${PN}-ptest += "coreutils procps bash bzip2 diffutils perl sed grep"
 RDEPENDS:${PN}-ptest += "e2fsprogs-badblocks e2fsprogs-dumpe2fs e2fsprogs-e2fsck e2fsprogs-mke2fs e2fsprogs-resize2fs e2fsprogs-tune2fs"
 
 do_compile_ptest() {
@@ -134,7 +133,8 @@ do_install_ptest() {
 	sed -e 's!../e2fsck/e2fsck!e2fsck!g' \
 	    -e 's!../misc/tune2fs!tune2fs!g' -i ${D}${PTEST_PATH}/test/*/expect*
 	sed -e 's!../e2fsck/e2fsck!${base_sbindir}/e2fsck!g' -i ${D}${PTEST_PATH}/test/*/script
-	sed -i "s#@PTEST_PATH@#${PTEST_PATH}#g" ${D}${PTEST_PATH}/test/test_script ${D}${PTEST_PATH}/test/test_one
+	sed -i -e 's#@PTEST_PATH@#${PTEST_PATH}#g' -e 's#^EGREP=.*#EGREP="grep -E"#' \
+        ${D}${PTEST_PATH}/test/test_script ${D}${PTEST_PATH}/test/test_one
 
 	# Remove various files
 	find "${D}${PTEST_PATH}" -type f \
-- 
2.49.0

Re: [OE-core] [PATCH 1/1] e2fsprogs: 1.47.1 -> 1.47.3

[Richard Purdie]

On Mon, 2025-07-21 at 06:48 -0700, Robert Yang via lists.openembedded.org wrote:
> From: Robert Yang <liezhi.yang@windriver.com>
> 
> * Remove backported patch 0001-libext2fs-fix-std-c23-build-failure.patch
> * Fixed QA Issue:
>   ERROR: e2fsprogs-1.47.2-r0 do_package_qa: QA Issue: File /usr/lib/e2fsprogs/ptest/test/test_one in package e2fsprogs-ptest contains reference to TMPDIR [buildpaths]
> 
> Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
> ---
>  ...-libext2fs-fix-std-c23-build-failure.patch | 42 -------------------
>  ...2fsprogs_1.47.1.bb => e2fsprogs_1.47.2.bb} |  8 ++--
>  2 files changed, 4 insertions(+), 46 deletions(-)
>  delete mode 100644 meta/recipes-devtools/e2fsprogs/e2fsprogs/0001-libext2fs-fix-std-c23-build-failure.patch
>  rename meta/recipes-devtools/e2fsprogs/{e2fsprogs_1.47.1.bb => e2fsprogs_1.47.2.bb} (96%)

There does look to be an issue with some hosts, e.g. debian 11:

https://autobuilder.yoctoproject.org/valkyrie/#/builders/35/builds/2008

Cheers,

Richard

Re: [OE-core] [PATCH 1/1] e2fsprogs: 1.47.1 -> 1.47.3

[Robert Yang]

Hi RP,

On 7/23/25 05:56, Richard Purdie wrote:
> On Mon, 2025-07-21 at 06:48 -0700, Robert Yang via lists.openembedded.org wrote:
>> From: Robert Yang <liezhi.yang@windriver.com>
>>
>> * Remove backported patch 0001-libext2fs-fix-std-c23-build-failure.patch
>> * Fixed QA Issue:
>>    ERROR: e2fsprogs-1.47.2-r0 do_package_qa: QA Issue: File /usr/lib/e2fsprogs/ptest/test/test_one in package e2fsprogs-ptest contains reference to TMPDIR [buildpaths]
>>
>> Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
>> ---
>>   ...-libext2fs-fix-std-c23-build-failure.patch | 42 -------------------
>>   ...2fsprogs_1.47.1.bb => e2fsprogs_1.47.2.bb} |  8 ++--
>>   2 files changed, 4 insertions(+), 46 deletions(-)
>>   delete mode 100644 meta/recipes-devtools/e2fsprogs/e2fsprogs/0001-libext2fs-fix-std-c23-build-failure.patch
>>   rename meta/recipes-devtools/e2fsprogs/{e2fsprogs_1.47.1.bb => e2fsprogs_1.47.2.bb} (96%)
> 
> There does look to be an issue with some hosts, e.g. debian 11:
> 
> https://autobuilder.yoctoproject.org/valkyrie/#/builders/35/builds/2008
> 

Got it, I'm on holiday, will fix it after come back next Monday.

> Cheers,
> 
> Richard