Re: [morty PATCH v2 1/3] ti-cgt-arm: add recipe for 16.9.2

[Karthik Ramanan <a0393906@ti.com>]

Denys,

I've taken care of your comments:
* renamed the recipe to include the version name.
* also made it native.

Posted v3.

Regards
Karthik

On 24-Apr-17 11:06 PM, Denys Dmytriyenko wrote:
> On Mon, Apr 24, 2017 at 06:37:41PM +0530, Karthik Ramanan wrote:
>> Jake, Denys - some questions for you.
>>
>>>> +PV = "16.9.2"
>>> Would it be better to have this in the recipe filename? I'm not
>> sure if there is a preference.
>> Denys, what is the recommendation? I see both approaches, I can
>> rework and submit if the PV should be included in the filename.
> Depends on how often you plan to update the recipe and whether you will have
> between-releases snapshots. That's usually the case with git-based components.
> In case of tarball or binary-installer components, the convention is to use
> the version in the filename.
>
>
>>>> +BBCLASSEXTEND = "native nativesdk"
>>> It appears that there is not support for the "target" class as the
>> installer is for the x86 architecture.
>> Just to be clear, would it be fine to just remove the nativesdk?
> Are there plans to have ARM installer? If not, then "target" recipe with
> BBCLASSEXTEND won't work - you just need to do a -native recipe directly.
>