Re: [OE-core] [PATCH] meta: simplify conditional operations with bb.utils.filter

[Joao Marcos Costa <joaomarcos.costa@bootlin.com>]

Hello again,

(...)
> 
> Yet another option is to do
> 
> (d.getVar('TUNE_CCARGS_MFPU') or '').strip() != ''
> 
> but it's a bit eww :)

I'm pretty sure that's how ERROR_QA is handled, actually:

meta/lib/oe/qa.py:    if error_class in (d.getVar("ERROR_QA") or 
"").split():

Unless this is different in a way I simply cannot see :)

> 
>> b) Simply remove those changes, then send another patch keeping only 
>> the changes in recipes.
>>
> 
> Yeah that works too. It's the same change but the implications are 
> different. So maybe three patches (one for machihes, one for QA, one for 
> recipes (maybe even in two, one for PACKAGECONFIG, one for RDEPENDS 
> since Chen Qi doesn't seem to like the latter too much :) ) so everybody 
> can bikeshed as much as they want on one and still have the rest merged :)
> 
> Cheers,
> Quentin

I'll do that. Maybe that's what I should have done in the first place, 
but I didn't want to spam N different patches that essentially do the 
same replacement.

Thanks!

-- 
Best regards,
João Marcos Costa