On Wed, Oct 22, 2008 at 12:48:02PM +0200, Stefan Reinauer wrote: > [EMAIL PROTECTED] wrote: > > Author: jcrouse > > Date: 2008-10-21 23:49:48 +0200 (Tue, 21 Oct 2008) > > New Revision: 3681 > > > > Modified: > > trunk/payloads/libpayload/Makefile > > trunk/payloads/libpayload/drivers/video/video.c > > trunk/payloads/libpayload/sample/hello.c > > Log: > > [PATCH] fix video console init > > > > Move console_add_output-driver() inside the for() loop > > > > Signed-off-by: Jordan Crouse <[EMAIL PROTECTED]> > > Acked-by: Jordan Crouse <[EMAIL PROTECTED]> > > > > > > Modified: trunk/payloads/libpayload/Makefile > > =================================================================== > > --- trunk/payloads/libpayload/Makefile 2008-10-21 16:27:38 UTC (rev > > 3680) > > +++ trunk/payloads/libpayload/Makefile 2008-10-21 21:49:48 UTC (rev > > 3681) > > @@ -115,6 +115,8 @@ > > $(Q)printf " AR $(subst $(shell pwd)/,,$(@))\n" > > $(Q)$(AR) rc $@ $(OBJS) > > > > +include util/kconfig/Makefile > > + > > $(obj)/%.o: $(src)/%.c > > $(Q)printf " CC $(subst $(shell pwd)/,,$(@))\n" > > $(Q)$(CC) -m32 $(CFLAGS) -c -o $@ $< > > @@ -164,7 +166,6 @@ > > $(Q)rm -rf build > > $(Q)rm -f .config .config.old ..config.tmp .kconfig.d .tmpconfig* > > > > -include util/kconfig/Makefile > > > > .PHONY: $(PHONY) prepare clean distclean doxygen doxy > > > > This part is not mentioned in the changelog of the commit. I assume it > has been committed by accident, because it breaks libpayload here: > > ra:libpayload stepan$ make menuconfig > make: *** No rule to make target `menuconfig'. Stop. > ra:libpayload stepan$ make config > make: *** No rule to make target `config'. Stop. > ra:libpayload stepan$
Yep, was committed accidentally I think. Moving that include as above is also not sufficient, I think, as the 'ifeq' part in the Makefile is not honored. Please try attached patch, works for me. I hope this will also fix Myles' compile problems. Uwe. -- http://www.hermann-uwe.de | http://www.holsham-traders.de http://www.crazy-hacks.org | http://www.unmaintained-free-software.org -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
