Just a heads-up, this is a stand-alone patch, not a series, despite the
subject line :(

Alex


On Wed, Dec 18, 2013 at 7:24 PM, Alex DAMIAN <[email protected]>wrote:

> From: Alexandru DAMIAN <[email protected]>
>
> Toaster needs buildstats to be enabled in order to
> collect task statistics. The toaster.bbclass didn't
> have a guard to stop task data collection if the buildstats
> weren't enabled, leading to a crash.
>
> This patch verifies that the task variables are defined before
> trying to use them.
>
> Signed-off-by: Alexandru DAMIAN <[email protected]>
> ---
>  meta/classes/toaster.bbclass | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/meta/classes/toaster.bbclass b/meta/classes/toaster.bbclass
> index e1a93b5..08c7bfa 100644
> --- a/meta/classes/toaster.bbclass
> +++ b/meta/classes/toaster.bbclass
> @@ -165,6 +165,9 @@ python toaster_collect_task_stats() {
>      import bb.utils
>      import os
>
> +    if None == e.data.getVar('BUILDSTATS_BASE', True):
> +        return  # if we don't have buildstats, we cannot collect stats
> +
>      def _append_read_list(v):
>          lock = bb.utils.lockfile(e.data.expand("${TOPDIR}/toaster.lock"),
> False, True)
>
> --
> 1.8.3.2
>
>


-- 
Alex Damian
Yocto Project
SSG / OTC
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Reply via email to