Re: [PATCH][yocto-autobuilder] lib/buildsteps.py: fix report creation in nigthly-oecore

[Beth 'pidge' Flanagan <pidge@toganlabs.com>]

On Fri, 2016-07-29 at 10:10 -0500, Aníbal Limón wrote:
> nightly-oecore buildset doesn't prove got_revision property
> so when got_revision is None use got_revision_oecore instead.
> 
> [YOCTO #10051]
> 
> Signed-off-by: Aníbal Limón <anibal.limon@linux.intel.com>
> ---
>  lib/python2.7/site-packages/autobuilder/lib/buildsteps.py | 7 +++++-
> -
>  1 file changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/lib/python2.7/site-
> packages/autobuilder/lib/buildsteps.py b/lib/python2.7/site-
> packages/autobuilder/lib/buildsteps.py
> index eee0c1b..37ac8b0 100644
> --- a/lib/python2.7/site-packages/autobuilder/lib/buildsteps.py
> +++ b/lib/python2.7/site-packages/autobuilder/lib/buildsteps.py
> @@ -118,8 +118,11 @@ class BitbakeShellCommand(ShellCommand):
>          report['target_sys'] = vrs['TARGET_SYS']
>  
>          report['component'] = 'bitbake'
> -        report['branch_commit'] = self.getProperty('branch') + ': '
> + \
> -                self.getProperty('got_revision')
> +        branch = self.getProperty('branch')
> +        revision = self.getProperty('got_revision')
> +        if not revision:
> +            revision = self.getProperty('got_revision_oecore')
> +        report['branch_commit'] = branch + ': ' + revision
> 

I'd put this in a try except. What happens with eclipse?

-b

>  
>          failure = {}
>          failure['package'] = "bitbake-%s" % vrs['BB_VERSION']
> -- 
> 2.1.4
>