[coreboot] Config.lb WTF
peter at stuge.se
Sun Apr 12 16:23:47 CEST 2009
Carl-Daniel Hailfinger wrote:
> >>> action "$(CC) $(DISTRO_CFLAGS) $(CFLAGS) $(CPPFLAGS) $(DEBUG_CFLAGS)
> >>> -I$(TOP)/src -I. -nostdinc -nostdlib -fno-builtin -Wall -Os -c -S
> >>> $(MAINBOARD)/$(CACHE_AS_RAM_AUTO_C) -o $@"
> > Since the config tool supports include files, why not exploit that
> > capability to fix this problem.
> And how do you determine which version you have to include? Guess? Hope?
> I'd prefer a systematic solution.
> 1. Unify the code as much as possible.
Didn't you just do this? Or almost? :)
> 2. Explain the remaining differences (and spot the bugs).
Are there any?
> 3. Automatic/scripted conversion to include files.
> > Longer term, we need a real set of Makefiles a la v3,
I completely agree.
> my 3-step plan is the only viable step forward.
In the short term, agreed. Please go ahead!
More information about the coreboot