I just checked. The problem could not be reproduced on my side. Commands and 
output are as below:

root@qemux86-64:/usr/lib/findutils/ptest# grep FAIL log
root@qemux86-64:/usr/lib/findutils/ptest# grep PASS log
PASS: tests/find/arg-nan.sh
PASS: tests/find/debug-missing-arg.sh
PASS: tests/find/exec-plus-last-file.sh
PASS: tests/find/execdir-fd-leak.sh
PASS: tests/find/files0-from.sh
PASS: tests/find/name-lbracket-literal.sh
PASS: tests/find/name-slash.sh
PASS: tests/find/newer.sh
PASS: tests/find/opt-numeric-arg.sh
PASS: tests/find/printf_escape_c.sh
PASS: tests/find/printf_escapechars.sh
PASS: tests/find/printf_inode.sh
PASS: tests/find/refuse-noop.sh
PASS: tests/find/type_list.sh
PASS: tests/find/used.sh
PASS: tests/find/user-group-max.sh
PASS: tests/misc/help-version.sh
PASS: tests/xargs/conflicting_opts.sh
PASS: tests/xargs/verbose-quote.sh
root@qemux86-64:/usr/lib/findutils/ptest# rpm
rpm        rpmbuild   rpmgraph   rpmlua     rpmsign    rpmspec
rpm2cpio   rpmdb      rpmkeys    rpmquery   rpmsort    rpmverify
root@qemux86-64:/usr/lib/findutils/ptest# rpm -qa | grep musl
musl-1.2.4+git0+79bdacff83-r0.core2_64

Regards,
Qi

-----Original Message-----
From: Khem Raj <[email protected]> 
Sent: Saturday, June 15, 2024 6:57 AM
To: Chen, Qi <[email protected]>
Cc: [email protected]
Subject: Re: [OE-core][PATCH 2/2] findutils: upgrade from 4.9.0 to 4.10.0

This fails ptests with qemux86_64/musl

bitbake core-image-ptest-findutils
results in

-find: invalid user name or UID argument to -user: '4294967296'
+find: invalid user name or UID argument to -user: ‘4294967296’
--- exp
+++ err
@@ -1 +1 @@
-find: invalid group name or GID argument to -group: '4294967296'
+find: invalid group name or GID argument to -group: ‘4294967296’
FAIL: tests/find/user-group-max.sh


On Thu, Jun 6, 2024 at 5:20 AM Chen Qi via lists.openembedded.org 
<[email protected]> wrote:
>
> From: Chen Qi <[email protected]>
>
> autoconf-2.73.patch is dropped as it's not needed any more.
>
> The run-ptest script is fixed. Instead copying function definitions 
> from init.cfg file, we should just install init.cfg file in the ptest 
> package. The coreutils-getlimits is added as a runtime dependency of 
> the ptest package because the getlimits command is needed.
>
> Signed-off-by: Chen Qi <[email protected]>
> ---
>  meta/recipes-extended/findutils/findutils.inc |  1 -
>  .../findutils/findutils/autoconf-2.73.patch   | 24 ----------------
>  .../findutils/findutils/run-ptest             | 28 -------------------
>  ...findutils_4.9.0.bb => findutils_4.10.0.bb} |  6 ++--
>  4 files changed, 3 insertions(+), 56 deletions(-)  delete mode 100644 
> meta/recipes-extended/findutils/findutils/autoconf-2.73.patch
>  rename meta/recipes-extended/findutils/{findutils_4.9.0.bb => 
> findutils_4.10.0.bb} (78%)
>
> diff --git a/meta/recipes-extended/findutils/findutils.inc 
> b/meta/recipes-extended/findutils/findutils.inc
> index 03cf2a8d72..ddcc05750b 100644
> --- a/meta/recipes-extended/findutils/findutils.inc
> +++ b/meta/recipes-extended/findutils/findutils.inc
> @@ -7,7 +7,6 @@ BUGTRACKER = "http://savannah.gnu.org/bugs/?group=findutils";
>  SECTION = "console/utils"
>
>  SRC_URI = "${GNU_MIRROR}/${BPN}/${BP}.tar.xz \
> -          file://autoconf-2.73.patch \
>            file://run-ptest \
>            "
>
> diff --git 
> a/meta/recipes-extended/findutils/findutils/autoconf-2.73.patch 
> b/meta/recipes-extended/findutils/findutils/autoconf-2.73.patch
> deleted file mode 100644
> index 63728b5f91..0000000000
> --- a/meta/recipes-extended/findutils/findutils/autoconf-2.73.patch
> +++ /dev/null
> @@ -1,24 +0,0 @@
> -The gnulib largefile macro needs updating to work with autoconf 2.73. 
> Rather -than the full code:
> -
> -https://git.savannah.gnu.org/cgit/gnulib.git/commit/m4/largefile.m4?i
> d=f91f633858cf132e50924224c50d6264a92caabb
> -
> -Just tweak the exiting code to work with 2.73. The next findutils 
> upgrade should -update to new gnulib
> -
> -Upstream-Status: Inappropriate
> -Signed-off-by: Richard Purdie <[email protected]>
> -
> -Index: findutils-4.9.0/gl/m4/largefile.m4
> -===================================================================
> ---- findutils-4.9.0.orig/gl/m4/largefile.m4
> -+++ findutils-4.9.0/gl/m4/largefile.m4
> -@@ -26,7 +26,7 @@ AC_DEFUN([gl_SET_LARGEFILE_SOURCE],
> - # with _TIME_BITS.  Also, work around a problem in autoconf <= 2.69:
> - # AC_SYS_LARGEFILE does not configure for large inodes on Mac OS X 
> 10.5,
> - # or configures them incorrectly in some cases.
> --m4_version_prereq([2.70], [], [
> -+m4_version_prereq([2.73], [], [
> -
> - # _AC_SYS_LARGEFILE_TEST_INCLUDES
> - # -------------------------------
> diff --git a/meta/recipes-extended/findutils/findutils/run-ptest 
> b/meta/recipes-extended/findutils/findutils/run-ptest
> index ec71583c51..3dec117067 100644
> --- a/meta/recipes-extended/findutils/findutils/run-ptest
> +++ b/meta/recipes-extended/findutils/findutils/run-ptest
> @@ -10,34 +10,6 @@ export built_programs="find xargs locate updatedb"
>  # this gets substituted by sed during build  export 
> VERSION="__run_ptest_version__"
>
> -# define missing functions for tests/init.sh -fu_path_prepend_ () {
> -       path_prepend_ $@
> -}
> -
> -print_ver_ () {
> -       :
> -}
> -
> -skip_if_root_ () {
> -       [ $(id -u) = 0 ] && exit 77;
> -}
> -
> -require_root_ () {
> -       [ $(id -u) = 0 ] || exit 77;
> -}
> -
> -expensive_ () {
> -       :
> -}
> -
> -export -f fu_path_prepend_
> -export -f print_ver_
> -export -f skip_if_root_
> -export -f require_root_
> -export -f expensive_
> -
> -
>  for f in tests/*/*.sh; do
>         bash $f ;
>         case $? in
> diff --git a/meta/recipes-extended/findutils/findutils_4.9.0.bb 
> b/meta/recipes-extended/findutils/findutils_4.10.0.bb
> similarity index 78%
> rename from meta/recipes-extended/findutils/findutils_4.9.0.bb
> rename to meta/recipes-extended/findutils/findutils_4.10.0.bb
> index 074817df27..57f77de16f 100644
> --- a/meta/recipes-extended/findutils/findutils_4.9.0.bb
> +++ b/meta/recipes-extended/findutils/findutils_4.10.0.bb
> @@ -6,7 +6,7 @@ LIC_FILES_CHKSUM = 
> "file://COPYING;md5=1ebbd3e34237af26da5dc08a4e440464"
>
>  DEPENDS = "bison-native"
>
> -SRC_URI[sha256sum] = 
> "a2bfb8c09d436770edc59f50fa483e785b161a3b7b9d547573cb08065fd462fe"
> +SRC_URI[sha256sum] = 
> "1387e0b67ff247d2abde998f90dfbf70c1491391a59ddfecb8ae698789f0a4f5"
>
>  PACKAGECONFIG[selinux] = "--with-selinux,--without-selinux,libselinux"
>  # http://savannah.gnu.org/bugs/?27299
> @@ -15,13 +15,13 @@ CACHED_CONFIGUREVARS += "gl_cv_func_wcwidth_works=yes"
>  EXTRA_OECONF += "ac_cv_path_SORT=${bindir}/sort"
>
>  # need od from coreutils for -t option -RDEPENDS:${PN}-ptest += "bash 
> sed grep coreutils"
> +RDEPENDS:${PN}-ptest += "bash sed grep coreutils coreutils-getlimits"
>
>  do_install_ptest:class-target() {
>         mkdir -p ${D}${PTEST_PATH}/tests/
> +       cp ${S}/init.cfg ${D}${PTEST_PATH}
>         cp -r ${S}/tests/* ${D}${PTEST_PATH}/tests/
>
>         # substitute value in run-ptest with actual version
>         sed -i -e 's/__run_ptest_version__/${PV}/' 
> ${D}${PTEST_PATH}/run-ptest
> -
>  }
> --
> 2.34.1
>
>
> 
>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#200664): 
https://lists.openembedded.org/g/openembedded-core/message/200664
Mute This Topic: https://lists.openembedded.org/mt/106521747/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to