On Thu Mar 19, 2026 at 6:55 PM CET, Joao Marcos Costa wrote:
> 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 <[email protected]>
>>> ---
>> 
>> 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!

Yes, I believe info or note are acceptable.

Thanks,
Mathieu


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

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#233547): 
https://lists.openembedded.org/g/openembedded-core/message/233547
Mute This Topic: https://lists.openembedded.org/mt/118399845/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to