[OE-core] [PATCH 1/1] oeqa/utils/commands.py: Fix get_bb_vars() when called without arguments

[Leonardo Sandoval]

On 12/14/2016 10:04 AM, Lopez, Mariano wrote:
>
>
> On 12/14/2016 10:01 AM, Leonardo Sandoval wrote:
>>
>>
>> On 12/14/2016 01:45 AM, mariano.lopez at linux.intel.com wrote:
>>> From: Mariano Lopez <mariano.lopez at linux.intel.com>
>>>
>>> Commit 9d55e9d489cd78be592fb9b4d6484f9060c62fdd broke calling 
>>> get_bb_vars()
>>> when called without arguments. This fix this issue.
>>>
>>> Signed-off-by: Mariano Lopez <mariano.lopez at linux.intel.com>
>>> ---
>>>   meta/lib/oeqa/utils/commands.py | 3 ++-
>>>   1 file changed, 2 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/meta/lib/oeqa/utils/commands.py 
>>> b/meta/lib/oeqa/utils/commands.py
>>> index 6acb24a..aecf8cf 100644
>>> --- a/meta/lib/oeqa/utils/commands.py
>>> +++ b/meta/lib/oeqa/utils/commands.py
>>> @@ -149,7 +149,8 @@ def get_bb_vars(variables=None, target=None, 
>>> postconfig=None):
>>>       """Get values of multiple bitbake variables"""
>>>       bbenv = get_bb_env(target, postconfig=postconfig)
>>>   -    variables = variables.copy()
>>> +    if variables is not None:
>>> +        variables = variables.copy()
>>
>> Is 'variables' type is  a dict (or some derived type)? I see some 
>> get_bb_env calls using lists and lists do not have the copy method.
>
> I only see 3 calls in OE core, two of them uses None as first 
> argument, and the last one uses a list, also if you check the function 
> it will handle the argument as a list; so the function expect a list 
> or None. And the list support the copy method, I just double check it:
>
> >>> l = [1,2,3]
> >>> l.copy()
> [1, 2, 3]

Sorry, I was using the python2 interpreter when I was doing this check. 
P3 included the copy method on the list data structure.



>
>