On Mon, 2026-04-27 at 15:08 +0200, Joao Marcos Costa via lists.openembedded.org wrote: > Some configuration files use bb.utils.contains to check for a string > inside a variable, and return the exact same string if true. > > This can be simplified by a call to bb.utils.filter, since the result is > the same, and the inline is shorter. > > Replace "bb.utils.contains(A, 'a', 'a', '', d)" by "bb.utils.filter(A, 'a', > d)". > > bb.utils.filter() does not return the string with a leading space, and > this is handled by a leading space outside of the helper. This > workaround, however, has its limitations: the leading space is always > added. To avoid any potential issues, use .strip() when dereferencing > TUNE_CCARGS_MFPU in the if statements. > > Signed-off-by: João Marcos Costa <[email protected]> > --- > meta/conf/machine/include/arm/feature-arm-neon.inc | 8 ++++---- > meta/conf/machine/include/arm/feature-arm-vfp.inc | 8 ++++---- > 2 files changed, 8 insertions(+), 8 deletions(-)
We've taken the other patches but not this one since we don't think this one is a net gain in readability or usability and there is no pressing reason we have to change. Cheers, Richard
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#236439): https://lists.openembedded.org/g/openembedded-core/message/236439 Mute This Topic: https://lists.openembedded.org/mt/119029956/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
