[yocto] [yocto-autobuilder][PATCHv2 1/5] buildsteps/BuildImages: Setproperty BUILDIMAGES

[Aníbal Limón]

On 07/03/2017 09:02 AM, Joshua Lock wrote:
> On Tue, 2017-06-06 at 10:46 -0500, Aníbal Limón wrote:
>> In certain situations is need to know what images was built for
>> example the DAFT executes flashing over certain image.
>>
>> [YOCTO #10604]
>>
>> Signed-off-by: Aníbal Limón <anibal.limon at linux.intel.com>
>> ---
>>  lib/python2.7/site-packages/autobuilder/buildsteps/BuildImages.py |
>> 3 +++
>>  1 file changed, 3 insertions(+)
>>
>> diff --git a/lib/python2.7/site-
>> packages/autobuilder/buildsteps/BuildImages.py b/lib/python2.7/site-
>> packages/autobuilder/buildsteps/BuildImages.py
>> index 30f7a18..7262fd3 100644
>> --- a/lib/python2.7/site-
>> packages/autobuilder/buildsteps/BuildImages.py
>> +++ b/lib/python2.7/site-
>> packages/autobuilder/buildsteps/BuildImages.py
>> @@ -146,6 +146,9 @@ class BuildImages(BitbakeShellCommand):
>>                  if self.minnowExists is None or self.minnowExists ==
>> "True":
>>                      self.command = ". ./oe-init-build-env; bitbake "
>> + bitbakeflags + self.images
>>                      self.description = ["Building " +
>> str(self.images)]
>> +        
>> +        self.setProperty("BUILDIMAGES", self.images)
> 
> Generally speaking caps case property names should only be used when
> they map to a variable in OE. BuildImages would work.

Agree,

Anibal
> 
>> +
>>          ShellCommand.start(self)
>>  
>>      def describe(self, done=False):