Re: [OE-core] [PATCH] ltp: fix warning in remove_broken_musl_sources

["Mathieu Dubois-Briand" <mathieu.dubois-briand@bootlin.com>]

On Thu Mar 19, 2026 at 2:29 PM CET, Joao Marcos Costa via lists.openembedded.org wrote:
> If this was originally intended to be an actual warning, it was not
> behaving as such: echo only prints to log.do_patch, so the message ends
> up hidden there.
>
> Replace 'echo' by bbwarn to display the warning message correctly.
>
> Signed-off-by: João Marcos Costa <joaomarcos.costa@bootlin.com>
> ---

Hi João,

Thanks for your patch.

>  meta/recipes-extended/ltp/ltp_20260130.bb | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/recipes-extended/ltp/ltp_20260130.bb b/meta/recipes-extended/ltp/ltp_20260130.bb
> index 75c5b8b7bd..36432c15b2 100644
> --- a/meta/recipes-extended/ltp/ltp_20260130.bb
> +++ b/meta/recipes-extended/ltp/ltp_20260130.bb
> @@ -126,7 +126,7 @@ remove_broken_musl_sources() {
>  	[ "${TCLIBC}" = "musl" ] || return 0
>  
>  	cd ${S}
> -	echo "WARNING: remove unsupported tests (until they're fixed)"
> +	bbwarn "remove unsupported tests (until they're fixed)"

So I believe this will add a WARNING on all musl builds, right? I'm not
sure we can really accept that.

Thanks,
Mathieu

-- 
Mathieu Dubois-Briand, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com