From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hans-Christian Egtvedt Date: Thu, 30 Oct 2008 12:57:38 +0100 Subject: [Buildroot] [PATCH 6/9] Ask the kernel source where it will installmodules In-Reply-To: <20081030105250.3ca61a3c@hcegtvedt> References: <20081016203220.15826.26173.stgit@localhost.localdomain> <20081016203246.15826.56394.stgit@localhost.localdomain> <20081030105250.3ca61a3c@hcegtvedt> Message-ID: <20081030125738.0489d0dd@hcegtvedt> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net On Thu, 30 Oct 2008 10:52:50 +0100 Hans-Christian Egtvedt wrote: > On Thu, 16 Oct 2008 22:32:46 +0200 > "Grant Likely" wrote: > > > From: Grant Likely > > > > 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://busybox.net/lists/buildroot/attachments/20081030/ff50e343/attachment.bin