Note that reporting this does not mean Chen has to reproduce or fix it. And
you Khem should make it clear in your report.
Ptests are expected to pass only in the standard glibc/gcc configuration
that the autobuilder tests.
Alex
On Sat 15. Jun 2024 at 5.45, Khem Raj via lists.openembedded.org <raj.khem=
gmail....@lists.openembedded.org> wrote:
>
>
> On Fri, Jun 14, 2024 at 8:41 PM Chen, Qi <qi.c...@windriver.com> 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 <raj.k...@gmail.com>
>> Sent: Saturday, June 15, 2024 6:57 AM
>> To: Chen, Qi <qi.c...@windriver.com>
>> Cc: openembedded-core@lists.openembedded.org
>> 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
>> <Qi.Chen=windriver....@lists.openembedded.org> wrote:
>> >
>> > From: Chen Qi <qi.c...@windriver.com>
>> >
>> > 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 <qi.c...@windriver.com>
>> > ---
>> > 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 <richard.pur...@linuxfoundation.org>
>> > -
>> > -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 (#200666):
https://lists.openembedded.org/g/openembedded-core/message/200666
Mute This Topic: https://lists.openembedded.org/mt/106521747/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
