* [PATCH] ltp: fix warning in remove_broken_musl_sources @ 2026-03-19 13:29 João Marcos Costa 2026-03-19 17:38 ` [OE-core] " Mathieu Dubois-Briand 0 siblings, 1 reply; 6+ messages in thread From: João Marcos Costa @ 2026-03-19 13:29 UTC (permalink / raw) To: openembedded-core; +Cc: thomas.petazzoni, João Marcos Costa 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> --- 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)" # sync with upstream # https://github.com/linux-test-project/ltp/blob/master/ci/alpine.sh#L33 -- 2.47.0 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [OE-core] [PATCH] ltp: fix warning in remove_broken_musl_sources 2026-03-19 13:29 [PATCH] ltp: fix warning in remove_broken_musl_sources João Marcos Costa @ 2026-03-19 17:38 ` Mathieu Dubois-Briand 2026-03-19 17:55 ` Joao Marcos Costa 0 siblings, 1 reply; 6+ messages in thread From: Mathieu Dubois-Briand @ 2026-03-19 17:38 UTC (permalink / raw) To: joaomarcos.costa, openembedded-core; +Cc: thomas.petazzoni 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 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [OE-core] [PATCH] ltp: fix warning in remove_broken_musl_sources 2026-03-19 17:38 ` [OE-core] " Mathieu Dubois-Briand @ 2026-03-19 17:55 ` Joao Marcos Costa 2026-03-19 18:17 ` Mathieu Dubois-Briand 0 siblings, 1 reply; 6+ messages in thread From: Joao Marcos Costa @ 2026-03-19 17:55 UTC (permalink / raw) To: Mathieu Dubois-Briand, openembedded-core; +Cc: thomas.petazzoni 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 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [OE-core] [PATCH] ltp: fix warning in remove_broken_musl_sources 2026-03-19 17:55 ` Joao Marcos Costa @ 2026-03-19 18:17 ` Mathieu Dubois-Briand 2026-03-20 9:16 ` Joao Marcos Costa 0 siblings, 1 reply; 6+ messages in thread From: Mathieu Dubois-Briand @ 2026-03-19 18:17 UTC (permalink / raw) To: Joao Marcos Costa, openembedded-core; +Cc: thomas.petazzoni 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 <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! Yes, I believe info or note are acceptable. Thanks, Mathieu -- Mathieu Dubois-Briand, Bootlin Embedded Linux and Kernel engineering https://bootlin.com ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [OE-core] [PATCH] ltp: fix warning in remove_broken_musl_sources 2026-03-19 18:17 ` Mathieu Dubois-Briand @ 2026-03-20 9:16 ` Joao Marcos Costa 2026-03-20 9:26 ` Richard Purdie 0 siblings, 1 reply; 6+ messages in thread From: Joao Marcos Costa @ 2026-03-20 9:16 UTC (permalink / raw) To: Mathieu Dubois-Briand, openembedded-core; +Cc: thomas.petazzoni Hello, On 3/19/26 19:17, Mathieu Dubois-Briand wrote: > 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 <joaomarcos.costa@bootlin.com> >>>> --- >>> (...)>>> >> Yep, it will. Would an INFO/NOTE be acceptable instead? >> >> Thanks! > > Yes, I believe info or note are acceptable. > > Thanks, > Mathieu I sent a v2 to fix that. One more thing, though: this same function uses an 'rm -rfv' and the "-f" option will not raise an error if the file is not there. Do you think this should be addressed? I would send this in a different patch, of course. Thanks! -- Best regards, João Marcos Costa ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [OE-core] [PATCH] ltp: fix warning in remove_broken_musl_sources 2026-03-20 9:16 ` Joao Marcos Costa @ 2026-03-20 9:26 ` Richard Purdie 0 siblings, 0 replies; 6+ messages in thread From: Richard Purdie @ 2026-03-20 9:26 UTC (permalink / raw) To: joaomarcos.costa, Mathieu Dubois-Briand, openembedded-core Cc: thomas.petazzoni On Fri, 2026-03-20 at 10:16 +0100, Joao Marcos Costa via lists.openembedded.org wrote: > Hello, > > On 3/19/26 19:17, Mathieu Dubois-Briand wrote: > > 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 <joaomarcos.costa@bootlin.com> > > > > > --- > > > > > (...)>>> > > > Yep, it will. Would an INFO/NOTE be acceptable instead? > > > > > > Thanks! > > > > Yes, I believe info or note are acceptable. > > > > Thanks, > > Mathieu > > I sent a v2 to fix that. > > One more thing, though: this same function uses an 'rm -rfv' and the > "-f" option will not raise an error if the file is not there. > > Do you think this should be addressed? I would send this in a different > patch, of course. Tasks need to be idempotent, we can run then multiple times with the same result. I didn't check where this is being called but the -f is probably fine. Cheers, Richard ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2026-03-20 9:27 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2026-03-19 13:29 [PATCH] ltp: fix warning in remove_broken_musl_sources João Marcos Costa 2026-03-19 17:38 ` [OE-core] " Mathieu Dubois-Briand 2026-03-19 17:55 ` Joao Marcos Costa 2026-03-19 18:17 ` Mathieu Dubois-Briand 2026-03-20 9:16 ` Joao Marcos Costa 2026-03-20 9:26 ` Richard Purdie
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox