[coreboot] Config.lb WTF

[Peter Stuge]

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.

Sure thing.


> > 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!


//Peter