[OE-core] [PATCH 2/3] toolchain-shar-extract.sh: better default install path for extensible SDK

[Christopher Larson]

On Thu, Aug 20, 2015 at 12:34 AM, Brendan Le Foll <brendan.le.foll at intel.com
> wrote:

> On Wed, Aug 19, 2015 at 06:52:49PM +0100, Christopher Larson wrote:
> >    On Wed, Aug 19, 2015 at 6:32 AM, <[1]brendan.le.foll at intel.com>
> wrote:
> >
> >    From: Brendan Le Foll <[2]brendan.le.foll at intel.com>
> >    Extensible SDK cannot be installed as root so by default offer to
> >    install it in
> >    user's home directory under distro/distro_version replacing the normal
> >    SDK
> >    version '+' char with a '_' as that's a restricted character for
> >    bitbake
> >    Signed-off-by: Brendan Le Foll <[3]brendan.le.foll at intel.com>
> >
> >    Why not use a metadata variable for the default, replacing with @@ as
> >    with the other metadata variables, rather than hardcoding the default
> >    in the .sh where it can't be easily overridden by the distro or user?
>
> '~' seemed like a good option for all SDKs/distros seemed like a
> decent 'default' to me and the rest is fairly customisable with
> SDK_TITLE which has to be set anyways. Or do you mean a different var?
>

I don't really see the point of making something "fairly customizable",
personally, when it's trivial to make it completely customizable by using a
metadata variable, the way we do for nearly everything else.
-- 
Christopher Larson
clarson at kergoth dot com
Founder - BitBake, OpenEmbedded, OpenZaurus
Maintainer - Tslib
Senior Software Engineer, Mentor Graphics
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20150820/76d19e9a/attachment-0002.html>