[yocto] [PATCH] Autobuilder.py: Fix triggered build bug

[Flanagan, Elizabeth]

On Thu, Aug 28, 2014 at 4:33 PM, California Sullivan
<californialsullivan at gmail.com> wrote:
> When a build was configured to use the TriggerBuilds build step, it would
> take on the buildslave of the last triggered build rather than its own.
>
> Signed-off-by: California Sullivan <california.l.sullivan at intel.com>
> ---
>  lib/python2.7/site-packages/autobuilder/Autobuilder.py | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/lib/python2.7/site-packages/autobuilder/Autobuilder.py
> b/lib/python2.7/site-packages/autobuilder/Autobuilder.py
> index a582a03..2e2e899 100644
> --- a/lib/python2.7/site-packages/autobuilder/Autobuilder.py
> +++ b/lib/python2.7/site-packages/autobuilder/Autobuilder.py
> @@ -58,7 +58,7 @@ class Autobuilder:
>              self.set_props = {}
>              self.repos = []
>              self.properties = []
> -            self.builders = None
> +            self.builders = self.parseBuilders(buildset)
>              self.schedprops.append(NestedParameter(name='ss_' + buildset,
> label="<h3><div class='trigger_heading' id='"+buildset+"'>" + buildset + "
> defaults:</div></h3>", fields=[FixedParameter(name="dummy",
> default="dummy")]))
>              self.parseBuildSet(buildset)
>              locals()['buildset_%s' %
> buildset]=BuildSet.BuildSet(name=buildset,
> @@ -86,7 +86,6 @@ class Autobuilder:
>
>      def parseBuildSet(self, buildset):
>          if buildset is not "BuildSets":
> -            self.builders=self.parseBuilders(buildset)
>              self.parseProps(buildset)
>              self.parseRepos(buildset)
>              self.parseSteps(buildset)
> --
> 1.9.1
>
>
> --
> _______________________________________________
> yocto mailing list
> yocto at yoctoproject.org
> https://lists.yoctoproject.org/listinfo/yocto
>


This looks ok, but I'd like to test it a bit before I shove it into
production. What have you tested this with (example: I tested it on a
single build triggering another one, or a build that triggers
itself...) With Trigger being a bit of core functionality, it would
help a bit to know this.

-b


-- 
Elizabeth Flanagan
Yocto Project
Build and Release