[yocto] [PATCH] kernel.bbclass: Iff exists, copy bounds.h for external module building.

[Bruce Ashfield]

On 12-07-30 11:07 AM, Bruce Ashfield wrote:
> On 12-07-28 09:45 AM, Leon Woestenberg wrote:
>> From: Leon Woestenberg<leon at sidebranch.com>
>>

Aha. And now I see that Scott G was copied. If this was for 1.2.x of yocto,
that needs to be clearly in the subject .. otherwise it is just
confusing.

My comment still stands that if this is needed for denzil, it should
be the same fixes we have in master, and not something different.

Cheers,

Bruce

>> Signed-off-by: Leon Woestenberg<leon at sidebranch.com>
>> ---
>> meta/classes/kernel.bbclass | 6 ++++++
>> 1 files changed, 6 insertions(+), 0 deletions(-)
>>
>> diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
>> index c21ab96..bd3e335 100644
>> --- a/meta/classes/kernel.bbclass
>> +++ b/meta/classes/kernel.bbclass
>> @@ -183,6 +183,12 @@ kernel_do_install() {
>> cp arch/powerpc/lib/crtsavres.o $kerneldir/arch/powerpc/lib/crtsavres.o
>> fi
>>
>> + # make -C $kerneldir _mrproper_scripts deleted this file, but we
>> need it for
>> + # building external modules like compat-wireless.
>> + if [ -f include/generated/bounds.h ]; then
>> + cp include/generated/bounds.h $kerneldir/include/generated/bounds.h
>> + fi
>
> This change is definitely safer using and older kernel that doesn't have a
> generated bounds.h .. but your tree is out of date. This should be on
> top of:
>
> commit 9247550642a536f83c6c711ce4bd340e5e5606b7
> Author: Denis Carikli <GNUtoo at no-log.org>
> Date: Fri Jul 20 16:22:22 2012 -0700
>
> kernel.bbclass: fix external module building
>
> Without that fix we have the following while compiling compat-wireless.
> include/linux/mmzone.h:18:30: fatal error: generated/bounds.h: No such
> file or directory
>
> Note that the compat-wireless recipe will be added in another commit.
>
> make -C $kerneldir _mrproper_scripts deleted this file along with
> other things so we resurrect it with this patch.
>
> (From OE-Core rev: 2d114fd0cdf7486f18e53f2bf41c4f559e4b4a8d)
>
> Signed-off-by: Denis 'GNUtoo' Carikli <GNUtoo at no-log.org>
> Signed-off-by: Koen Kooi <koen at dominion.thruhere.net>
> Signed-off-by: Saul Wold <sgw at linux.intel.com>
> Signed-off-by: Richard Purdie <richard.purdie at linuxfoundation.org>
>
>
> Cheers,
>
> Bruce
>
>> +
>> # Remove the following binaries which cause strip errors
>> # during do_package for cross-compiled platforms
>> bin_files="arch/powerpc/boot/addnote arch/powerpc/boot/hack-coff \
>
> _______________________________________________
> yocto mailing list
> yocto at yoctoproject.org
> https://lists.yoctoproject.org/listinfo/yocto