[yocto] Error during adding a new recipe.

[Khem Raj]

On Fri, Jul 22, 2016 at 12:09 AM, Andre McCurdy <armccurdy at gmail.com> wrote:
> On Thu, Jul 21, 2016 at 8:20 PM, Khem Raj <raj.khem at gmail.com> wrote:
>>
>>> On Jul 21, 2016, at 6:45 PM, Paul Eggleton <paul.eggleton at linux.intel.com> wrote:
>>>
>>> On Thu, 21 Jul 2016 18:35:25 Khem Raj wrote:
>>>> On Thu, Jul 21, 2016 at 6:32 PM, Paul Eggleton
>>>>
>>>> <paul.eggleton at linux.intel.com> wrote:
>>>>> On Thu, 21 Jul 2016 18:22:32 Khem Raj wrote:
>>>>>>> On Jul 21, 2016, at 5:20 PM, Vijayakumar Badiger
>>>>>>> <vijayakumarmb at gmail.com>
>>>>>>> wrote:
>>>>>>>
>>>>>>> The sync driver is already ported in the kernel.  I just need to get
>>>>>>> this
>>>>>>> compilation error fixed. I tried adding below change to kernel bb file
>>>>>>> but still I get that same error.
>>>>>>>
>>>>>>> PACKAGES =+ "kernel-headers"
>>>>>>> FILES_kernel-headers = "${KDIR}/usr/include”
>>>>>>
>>>>>> there is a different recipe for headers linux-libc-headers that also
>>>>>> needs
>>>>>> to be patched and then hopefully kernel build system is exporting this
>>>>>> header for userspace automatically and you will be set.
>>>>>
>>>>> I thought we weren't supposed to be encouraging modifying
>>>>> linux-libc-headers - especially as this isn't for the libc - it's for
>>>>> other userspace code...?
>>>>
>>>> Yes thats correct thats why
>>>> I am not asking for submitting this patch upstream.
>>>
>>> Sure, but this question comes up a lot. We ought to be telling people the
>>> correct mechanism for introducing extra headers such as this if it isn't to
>>> modify linux-libc-headers.
>>
>> Thats possible for external kernel modules. This is a kernel patch. There is no other elegant way
>> of solving this. They could write another recipe just to stage this header. but thats equally ugly.
>
> A new recipe isn't needed. You can add something like the following to
> the existing kernel recipe:
>
>   sysroot_stage_all_append () {
>     mkdir -p ${SYSROOT_DESTDIR}/${includedir}/linux
>     install -m644 ${S}/include/linux/sync.h
> ${SYSROOT_DESTDIR}/${includedir}/linux/
>   }

I wonder if this will this work with sstate

>
> and then add a dependency on the kernel to the recipe which needs the headers.
>
>
>>>
>>> Cheers,
>>> Paul
>>>
>>> --
>>>
>>> Paul Eggleton
>>> Intel Open Source Technology Centre
>>
>>
>> --
>> _______________________________________________
>> yocto mailing list
>> yocto at yoctoproject.org
>> https://lists.yoctoproject.org/listinfo/yocto
>>