All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] bitbake.conf: Add timeout to HOSTTOOLS
@ 2026-08-07 10:02 Hemanth.KumarMD
  2026-08-10  9:27 ` [OE-core] " Alexander Kanavin
  0 siblings, 1 reply; 3+ messages in thread
From: Hemanth.KumarMD @ 2026-08-07 10:02 UTC (permalink / raw)
  To: openembedded-core; +Cc: Sundeep.Kokkonda, Hemanth.KumarMD

From: Hemanth Kumar M D <Hemanth.KumarMD@windriver.com>

The glibc iconv/tst-iconv_prog test uses timeout(1) on the host to
guard against hangs during charset conversion testing. Since timeout
is not listed in HOSTTOOLS, it is unavailable in the Yocto build
environment, causing the test to fail with exit code 127 (command not
found) instead of the expected conversion error code.

AB testresults (qemux86-64-tc):

  +--------------+--------+--------+------+
  | Result       | Before | After  | Diff |
  +--------------+--------+--------+------+
  | PASS         | 6759   | 6760   | +1   |
  | FAIL         | 30     | 29     | -1   |
  | UNSUPPORTED  | 584    | 584    |  0   |
  | XFAIL        | 16     | 16     |  0   |
  +--------------+--------+--------+------+

  iconv/tst-iconv_prog  FAIL -> PASS

With this fix, iconv/tst-iconv_prog now passes on all architectures.
Verified on AB.

Signed-off-by: Hemanth Kumar M D <Hemanth.KumarMD@windriver.com>
---
 meta/conf/bitbake.conf | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/conf/bitbake.conf b/meta/conf/bitbake.conf
index bdf37d0da2..39966d60ef 100644
--- a/meta/conf/bitbake.conf
+++ b/meta/conf/bitbake.conf
@@ -533,7 +533,7 @@ HOSTTOOLS += " \
     mktemp mv nm objcopy objdump od patch perl pr printf pwd \
     python3 pzstd ranlib readelf readlink realpath rm rmdir rpcgen sed seq sh \
     sha1sum sha224sum sha256sum sha384sum sha512sum \
-    sleep sort split stat strings strip tail tar tee test touch tr true truncate uname \
+    sleep sort split stat strings strip tail tar tee test timeout touch tr true truncate uname \
     uniq unzstd wc wget which xargs zstd xz \
 "
 
-- 
2.49.0



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [OE-core] [PATCH] bitbake.conf: Add timeout to HOSTTOOLS
  2026-08-07 10:02 [PATCH] bitbake.conf: Add timeout to HOSTTOOLS Hemanth.KumarMD
@ 2026-08-10  9:27 ` Alexander Kanavin
  2026-08-10 10:05   ` Richard Purdie
  0 siblings, 1 reply; 3+ messages in thread
From: Alexander Kanavin @ 2026-08-10  9:27 UTC (permalink / raw)
  To: Hemanth.KumarMD; +Cc: openembedded-core, Sundeep.Kokkonda

On Fri, 7 Aug 2026 at 12:03, Hemanth Kumar M D via
lists.openembedded.org
<Hemanth.KumarMD=windriver.com@lists.openembedded.org> wrote:
> The glibc iconv/tst-iconv_prog test uses timeout(1) on the host to
> guard against hangs during charset conversion testing. Since timeout
> is not listed in HOSTTOOLS, it is unavailable in the Yocto build
> environment, causing the test to fail with exit code 127 (command not
> found) instead of the expected conversion error code.
> --- a/meta/conf/bitbake.conf
> +++ b/meta/conf/bitbake.conf
> @@ -533,7 +533,7 @@ HOSTTOOLS += " \
>      mktemp mv nm objcopy objdump od patch perl pr printf pwd \
>      python3 pzstd ranlib readelf readlink realpath rm rmdir rpcgen sed seq sh \
>      sha1sum sha224sum sha256sum sha384sum sha512sum \
> -    sleep sort split stat strings strip tail tar tee test touch tr true truncate uname \
> +    sleep sort split stat strings strip tail tar tee test timeout touch tr true truncate uname \
>      uniq unzstd wc wget which xargs zstd xz \
>  "

Adding to hosttools just to fix a single test is very excessive.
Rather make it depend on coreutils-native.

Alex


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [OE-core] [PATCH] bitbake.conf: Add timeout to HOSTTOOLS
  2026-08-10  9:27 ` [OE-core] " Alexander Kanavin
@ 2026-08-10 10:05   ` Richard Purdie
  0 siblings, 0 replies; 3+ messages in thread
From: Richard Purdie @ 2026-08-10 10:05 UTC (permalink / raw)
  To: alex.kanavin, Hemanth.KumarMD; +Cc: openembedded-core, Sundeep.Kokkonda

On Mon, 2026-08-10 at 11:27 +0200, Alexander Kanavin via lists.openembedded.org wrote:
> On Fri, 7 Aug 2026 at 12:03, Hemanth Kumar M D via
> lists.openembedded.org
> <Hemanth.KumarMD=windriver.com@lists.openembedded.org> wrote:
> > The glibc iconv/tst-iconv_prog test uses timeout(1) on the host to
> > guard against hangs during charset conversion testing. Since timeout
> > is not listed in HOSTTOOLS, it is unavailable in the Yocto build
> > environment, causing the test to fail with exit code 127 (command not
> > found) instead of the expected conversion error code.
> > --- a/meta/conf/bitbake.conf
> > +++ b/meta/conf/bitbake.conf
> > @@ -533,7 +533,7 @@ HOSTTOOLS += " \
> >      mktemp mv nm objcopy objdump od patch perl pr printf pwd \
> >      python3 pzstd ranlib readelf readlink realpath rm rmdir rpcgen sed seq sh \
> >      sha1sum sha224sum sha256sum sha384sum sha512sum \
> > -    sleep sort split stat strings strip tail tar tee test touch tr true truncate uname \
> > +    sleep sort split stat strings strip tail tar tee test timeout touch tr true truncate uname \
> >      uniq unzstd wc wget which xargs zstd xz \
> >  "
> 
> Adding to hosttools just to fix a single test is very excessive.
> Rather make it depend on coreutils-native.

I'm a bit torn on this. Since timeout is part of coreutils, the host
dependencies are basically unchanged by the patch since we already
require a good chunk of coreutils.

I do wish we could make "per recipe" markup of things like this work in
a reasonable way though as it is sad to mark this as needed
everywhere...

Cheers,

Richard


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2026-08-10 10:05 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-08-07 10:02 [PATCH] bitbake.conf: Add timeout to HOSTTOOLS Hemanth.KumarMD
2026-08-10  9:27 ` [OE-core] " Alexander Kanavin
2026-08-10 10:05   ` Richard Purdie

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.