[Toaster] general solution proposed for 7394 "Toaster needs to add a space to the value of IMAGE_INSTALL_append"

Reyna, David david.reyna at windriver.com
Thu Mar 12 07:54:56 PDT 2015


> With what we have left in the release, I rather
> go for the simplest acceptable solution.

Ok. We can do the full solution in YP-1.9.

However, that leaves us with "IMAGE_INSTALL_append" being still semantically wrong, and not a good foundation for the future.

If we can rename it to "IMAGE_INSTALL_+=" (following my general proposal), and make this single hardcoded exception, then we are talking a minimal change (a dozen lines of code).

The other option is to just call it "IMAGE_INSTALL", with no syntax sugar, and just make it work as a "+=" hardcoded in the background.

- David

> -----Original Message-----
> From: Barros Pena, Belen [mailto:belen.barros.pena at intel.com]
> Sent: Thursday, March 12, 2015 7:44 AM
> To: DAMIAN, ALEXANDRU; Reyna, David; Paul Eggleton; PURDIE, RICHARD
> Cc: toaster at yoctoproject.org
> Subject: Re: general solution proposed for 7394 "Toaster needs to add a
> space to the value of IMAGE_INSTALL_append"
> 
> On 12/03/2015 14:09, "Damian, Alexandru" <alexandru.damian at intel.com>
> wrote:
> 
> >In fact what we can do is have a pull down list in the page to specific
> >and select any of the supported special operators, and let the javascript
> >manipulate these special syntaxes. That way no one needs to
> >memorize/learn about the supported operators, making the whole thing more
> >intuitive and self-directed.
> 
> If we want to introduce operators, we'll need to provide controls in the
> configuration variables page to set them, together with help to explain
> what each of them means. Considering that we are implementing special
> controls to ease the manipulation of certain variables, I would need to
> find a way of inserting the operator control so that it works in all cases
> (and not only when we simply have a text field to set the variable value).
> 
> 
> I raised this question when we were designing the variables page, and the
> answer was not to include operators. I would rather not do it now at this
> late stage, to be honest. With what we have left in the release, I rather
> go for the simplest acceptable solution.
> 
> Cheers
> 
> Belén
> 
> 



More information about the toaster mailing list