Re: [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.
> Unify the code as much as possible.

Didn't you just do this? Or almost? :)
> Explain the remaining differences (and spot the bugs).

Are there any?
> 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