Hi Adrian, On Sun, Mar 01, 2026 at 03:20:55PM +0100, [email protected] wrote: > Hi Francesco > > Thank you for the nice patches. Also thank you for separating this > change into a separate patch to make it more obvious that this is > potentially a bit controversial. > > This looks like a risky change to me. Having a variable set to None by > accident is a common error which the test should catch. Just allowing > that is probably not the right approach. >
I agree with you TBH. > Is it really not possible to make the tests providing the correct > reference values in the dict which compared against the output? > After some more thinking, I think I got the way in which uboot-config.bbclass (which was recently reworked to support flags on UBOOT_CONFIG_* variables) is approaching the matter. I'll rework the test approach and send a v2. > Regards, > Adrian > Thank you! Regards, Francesco
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#232144): https://lists.openembedded.org/g/openembedded-core/message/232144 Mute This Topic: https://lists.openembedded.org/mt/118051540/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
