Hi Dan,

On 11/2/20 4:18 PM, Dan Murphy wrote:
> Suman
> 
> On 10/27/20 2:11 PM, Denys Dmytriyenko wrote:
>> On Tue, Oct 27, 2020 at 09:37:18AM -0500, Suman Anna wrote:
>>> Hi Denys,
>>>
>>> On 10/26/20 11:08 PM, Denys Dmytriyenko wrote:
>>>> On Mon, Oct 26, 2020 at 09:48:26PM -0500, Suman Anna wrote:
>>>>> On 10/26/20 9:45 PM, Denys Dmytriyenko wrote:
>>>>>> On Mon, Oct 26, 2020 at 09:41:12PM -0500, Suman Anna wrote:
>>>>>>> Add the ti-rpmsg-char dev packages to make the public headers available 
>>>>>>> in
>>>>>>> the target fileystem so that applications can be compiled natively on 
>>>>>>> the
>>>>>>> target.
>>>>>> -sdk-target is not for the target filesystem!
>>>>>> It is for the target sysroot in the devkit.
>>>>> Hmm, is this supposed to go into a different file or my patch description
>>>>> needs
>>>>> correcting?
>>>> It depends on what you actually need.
>>> I want the files from the  ti-rpmsg-char-dev package (the public header 
>>> files
>>> and libti_rpmsg_char.so) to be part of the FS tarball.
>> Then you need to pull corresponding -dev package into the packagegroup that
>> builds up the rootfs.
>>
>> At least in Arago, I kept the namespace convention that -sdk-target and
>> -sdk-host suffixes are used to identify packagegroups for the devkit -
>> cross-compile development environment on the host, target and host sysroots.
>>
>>> regards
>>> Suman
>>>
>>>>
>>>>>>> Signed-off-by: Suman Anna <[email protected]>
>>>>>>> ---
>>>>>>>   .../packagegroup-arago-tisdk-addons-sdk-target.bb         | 8 ++++++--
>>>>>>>   1 file changed, 6 insertions(+), 2 deletions(-)
>>>>>>>
>>>>>>> diff --git
>>>>>>> a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-target.bb
>>>>>>> b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-target.bb
>>>>>>>
>>>>>>> index e372dd67744e..b55c79a4d4df 100644
>>>>>>> ---
>>>>>>> a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-target.bb
>>>>>>>
>>>>>>> +++
>>>>>>> b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-addons-sdk-target.bb
>>>>>>>
>>>>>>> @@ -1,6 +1,6 @@
>>>>>>>   DESCRIPTION = "Task to install headers and libraries related to addons
>>>>>>> into the SDK"
>>>>>>>   LICENSE = "MIT"
>>>>>>> -PR = "r43"
>>>>>>> +PR = "r44"
>>>>>>>     PACKAGE_ARCH = "${MACHINE_ARCH}"
>>>>>>>   @@ -143,7 +143,11 @@ UTILS_append_dra7xx = " can-utils-dev \
>>>>>>>                           elfutils-staticdev \
>>>>>>>   "
>>>>>>>   UTILS_append_k2g = " can-utils-dev"
>>>>>>> -UTILS_append_k3 = " can-utils-dev"
>>>>>>> +UTILS_append_k3 = "\
>>>>>>> +    can-utils-dev \
>>>>>>> +    ti-rpmsg-char-dev \
>>>>>>> +    ti-rpmsg-char-staticdev \
>>>>>>> +"
>>>>>>>     EXTRA_LIBS = ""
>>>>>>>   EXTRA_LIBS_append_ti43x = "\
>>>>>>> -- 
>>>>>>> 2.28.0
>>>>>>>
> 
> Does this need to go in or are you going to send in v2?

I am confused by the terminology above, I am not sure if I need to move this to
the packagegroup-arago-tisdk-addons.bb file, or if this is just a question of
writing the proper patch description. I cannot verify myself until I fix my OE
build issues.

regards
Suman

> 
> Dan
> 
> 

_______________________________________________
meta-arago mailing list
[email protected]
http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

Reply via email to