[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