On Sat, 2024-03-09 at 02:29 -0800, Robert P. J. Day wrote:
>
> As ptest.bbclass defines the more intuitive ptest-related variable:
>
> PTEST_ENABLED =
> "${@bb.utils.contains('DISTRO_FEATURES', 'ptest', '1', '0', d)}"
>
> modify a number of obvious recipe ptest checks to use that simpler
> form.
>
> Signed-off-by: Robert P. J. Day <rpj...@crashcourse.ca>
>
> ---
>
> i did a simple textual substitution so as long as that's all that's
> needed, this should work.
As others have noted, it breaks things. This is why I'm very nervous of
patches that aren't properly tested :(.
I personally also really don't "1" being a magic value for comparison
but that is a different problem.
Cheers,
Richard
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#196974):
https://lists.openembedded.org/g/openembedded-core/message/196974
Mute This Topic: https://lists.openembedded.org/mt/104825491/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
