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

[Joao Marcos Costa <joaomarcos.costa@bootlin.com>]

Hello,

On 3/19/26 18:38, Mathieu Dubois-Briand wrote:
> 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.
>
Yep, it will. Would an INFO/NOTE be acceptable instead?

Thanks!

-- 
Best regards,
João Marcos Costa