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

[Joshua Lock <joshua.g.lock@linux.intel.com>]

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@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.

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