On Fri, Jun 14, 2024 at 8:41 PM Chen, Qi <[email protected]> wrote:

> 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
>

I have musl updated staged in contrib/yoe/mut I wonder if it’s seen with
newer musl I also use clang and libc++ so could be that too

>
> 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 (#200665): 
https://lists.openembedded.org/g/openembedded-core/message/200665
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