Re: [PATCH] kernel.bbclass: restore kernel-abiversion file

[Darren Hart <dvhart@linux.intel.com>]

On 06/22/2011 08:41 AM, Koen Kooi wrote:
> This fixes external module recipes that need $KERNEL_VERSION. It got removed by
> 
>     commit a9d41062e24a6b99661b3a5256f369b557433607
>     Author: Darren Hart <dvhart@linux.intel.com>
>     Date:   Tue Mar 8 17:09:10 2011 -0800
> 
>         kernel/bbclass: rework kernel and module classes to allow for building out-of-tree modules
> 
> seemingly as an oversight.
> 
> Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
> ---
>  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 1bb69f5..8f9f108 100644
> --- a/meta/classes/kernel.bbclass
> +++ b/meta/classes/kernel.bbclass
> @@ -123,6 +123,12 @@ kernel_do_install() {
>  	install -d $kerneldir
>  
>  	#
> +	# Store the kernel version in sysroots for module-base.bbclass
> +	#
> +
> +	echo "${KERNEL_VERSION}" > $kerneldir/kernel-abiversion

From everything I can see this is an openembedded construct - which
external modules need this? An example recipe I can review?

I'd like to understand this better as I don't like the idea of depending
on an arbitrary file that isn't part of the kernel build system to build
modules.

It seems to me that these modules should be able to DEPEND on something
provided by the kernel recipes, rather than checking the contents of a
file stuck in the build tree.

--
Darren

> +
> +	#
>  	# Copy the entire source tree. In case an external build directory is
>  	# used, copy the build directory over first, then copy over the source
>  	# dir. This ensures the original Makefiles are used and not the

-- 
Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel