[Buildroot] [PATCH 6/9] Ask the kernel source where it will installmodules

Hans-Christian Egtvedt hans-christian.egtvedt at atmel.com
Thu Oct 30 11:57:38 UTC 2008


On Thu, 30 Oct 2008 10:52:50 +0100
Hans-Christian Egtvedt <hans-christian.egtvedt at atmel.com> wrote:

> On Thu, 16 Oct 2008 22:32:46 +0200
> "Grant Likely" <grant.likely at secretlab.ca> wrote:
> 
> > From: Grant Likely <grant.likely at secretlab.ca>
> > 
> > Originally, $(LINUX26_VERSION) was used to figure out which
> > directory modules would get installed in under /lib/modules.  This
> > causes problems when $(LINUX26_VERSION) differs from what version
> > the kernel source thinks it is using because it will try to use
> > depmod in the wrong directory.
> > 
> 
> I like this stuff, I'll give it a spin and see how it performs. If it
> works well I'll add it to the repo (unless somebody beat me to it).
> 

Not present, and the stuff you posted was a little buggy. I cleaned it
up a bit, see the attached patch.

What do people think? I'll add it today/tomorrow unless somebody
objects.

> > This patch adds a new variable $(LINUX26_VERSION_MOD) which resolves
> > to the exact version string that the kernel uses.  Module install
> > then uses this value instead of $(LINUX26_VERSION) when running
> > depmod
> > 
> > This patch also eliminates the modules.dep target and makes
> > .modules_installed depend on $(LINUX26_DIR)/.configured directly.
> > This is necessary because the value of $(LINUX26_VERSION_MOD) is
> > unavailable before the kernel is unpacked.  Trying to use it in a
> > target or dependency name confuses make.
> 
> And what about using the $(PROJECT_DIR)/autotools-stamp directory
> instead for the depend file? Follow with the stream, etc... Easier to
> know how to trigger a reinstall as well.
> 

I also added this, since I like gathering all the stamp files at one
place for the target installations.

-- 
Best regards,
Hans-Christian Egtvedt
-------------- next part --------------
A non-text attachment was scrubbed...
Name: linux26.advanced-probe-kernel-version-before-installing-modules.patch
Type: text/x-patch
Size: 2119 bytes
Desc: not available
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20081030/ff50e343/attachment-0003.bin>


More information about the buildroot mailing list