On Mon, Feb 8, 2016 at 2:56 PM, Burton, Ross <[email protected]> wrote: > > On 8 February 2016 at 20:34, Andre McCurdy <[email protected]> wrote: >> >> '' if incompatible_license_contains('GPLv3', 'x', '', d) == 'x' or >> bb.utils.contains('DISTRO_FEATURES', 'x11', 'x', '', d) == '' else 'gnome' } >> \ > > Instead of returning 'x' or '' and then doing string comparisons, why not > just return True or False and let if/or/else do the right thing?
Just trying to preserve the original logic to make it clearer that no functionality is being changed. The logic can be traced back to Paul Eggleton's original version here: http://git.openembedded.org/openembedded-core/commit/?id=973351cf2e338f07c11395d494193e9c62857f2d > Ross -- _______________________________________________ Openembedded-core mailing list [email protected] http://lists.openembedded.org/mailman/listinfo/openembedded-core
