[Toaster] [PATCH] toaster: projectpage Make sure build targets are space separated

[Barros Pena, Belen]

On 17/11/2015 11:57, "toaster-bounces at yoctoproject.org on behalf of
Michael Wood" <toaster-bounces at yoctoproject.org on behalf of
michael.g.wood at intel.com> wrote:

>Make sure the build targets are space separated when building multiple
>targets. Also fix error path now that YOCTO #7995 is resolved.
>
>[YOCTO #8450]

This works for me. Thanks!

Belén

>
>Signed-off-by: Michael Wood <michael.g.wood at intel.com>
>---
> .../toaster/toastergui/static/js/projectpage.js    | 22
>+++++++++++++---------
> 1 file changed, 13 insertions(+), 9 deletions(-)
>
>diff --git a/bitbake/lib/toaster/toastergui/static/js/projectpage.js
>b/bitbake/lib/toaster/toastergui/static/js/projectpage.js
>index ae08d9a..8cbfa9e 100644
>--- a/bitbake/lib/toaster/toastergui/static/js/projectpage.js
>+++ b/bitbake/lib/toaster/toastergui/static/js/projectpage.js
>@@ -225,17 +225,21 @@ function projectPageInit(ctx) {
> 
>     var toBuild = "";
>     freqBuildList.find(":checked").each(function(){
>-      toBuild += $(this).val();
>+      toBuild += $(this).val() + ' ';
>     });
> 
>-    libtoaster.startABuild(libtoaster.ctx.projectBuildsUrl,
>libtoaster.ctx.projectId, toBuild, function(){
>-      /* Build started */
>-      window.location.replace(libtoaster.ctx.projectBuildsUrl);
>-    },
>-    function(){
>-      /* Build start failed */
>-      /* [YOCTO #7995] */
>-      window.location.replace(libtoaster.ctx.projectBuildsUrl);
>+    toBuild = toBuild.trim();
>+
>+    libtoaster.startABuild(libtoaster.ctx.projectBuildsUrl,
>+      libtoaster.ctx.projectId,
>+      toBuild,
>+      function(){
>+        /* Build request started */
>+       window.location.replace(libtoaster.ctx.projectBuildsUrl);
>+      },
>+      function(){
>+        /* Build request failed */
>+        console.warn("Build request failed to be created");
>     });
>   });
> 
>-- 
>2.5.0
>
>-- 
>_______________________________________________
>toaster mailing list
>toaster at yoctoproject.org
>https://lists.yoctoproject.org/listinfo/toaster