On 25/04/08 02:46 +0200, Uwe Hermann wrote:
On Thu, Apr 24, 2008 at 06:22:00PM -0600, Jordan Crouse wrote:
On 25/04/08 02:06 +0200, Uwe Hermann wrote:
See patch.
The actual patch is buildrom_update_kconfig.patch.gz but that's huge and totally unreadable so I recommend reviewing buildrom_update_kconfig_makefile.patch
That's the only change to the buildrom code itself, the rest is just replacing the whole scripts/kconfig/ dir with the new kconfig.
I used the coreinfo kconfig/ with only a handful of string replacements a la s/coreinfo/buildrom/ so the diff to coreinfo's kconfig is tiny.
I did some ad-hoc testing with this, tried various kconfig targets (menuconfig, xconfig, gconfig, ...) and various buildrom builds and it seems to work fine so far, but some more testing by others is highly appreciated.
+HOSTCC = gcc +HOSTCXX = g++ +HOSTCFLAGS := -I$(srck) -I$(objk) +HOSTCXXFLAGS := -I$(srck) -I$(objk)
Can you move these to scripts/Build.settings, please? Thats the common repository for these sorts of things. In fact, HOSTCC is already there.
Not easily I think, these variables are used by kconfig internally, but scripts/Build.settings is (currently) only read after the 'make config' or 'make menuconfig' step so that likely won't work.
Okay, then kill off the HOSTCC in Build.settings - this is very unfortunate, I would much rather keep this sort of thing out of the Makefile proper.
Jordan