On Fri, Oct 30, 2009 at 9:29 AM, Peter Stuge <[email protected]> wrote:

> Myles Watson wrote:
> > >> That functionality is already there... see util/xcompile.
> > >
> > > So what's the preferred way to invoke it?
>
> I think Makefile picks it up automatically if there is an .xcompile
> file in the right spot.<http://www.coreboot.org/mailman/listinfo/coreboot>
>

Thanks.  My problems getting it to work were:

I hadn't realized that the path to crossgcc was hard coded in xcompile, so
xcompile has to be invoked from the right spot:

for gccprefixes in `pwd`/../crossgcc/xgcc/bin/i386-elf- i386-elf- ""; do

I had expected to be able to run xcompile from the top like this:
. ./util/xcompile/xcompile > .xcompile
the way it works is:
cd util/xcompile
. ./xcompile > ../../.xcompile

The other problem I had was that the Makefile removes .xcompile on make
clean, which is not what I wanted.  I'd just generated the .xcompile file,
so I did a make clean to make sure the whole thing was compiled with the
same compiler.

I think it would be better to remove .xcompile with make distclean.  Patch
attached.

Signed-off-by: Myles Watson <[email protected]>

Thanks,
Myles
Index: svn/Makefile
===================================================================
--- svn.orig/Makefile
+++ svn/Makefile
@@ -289,7 +289,7 @@ doxygen-clean:
 	rm -rf $(DOXYGEN_OUTPUT_DIR)
 
 clean: doxygen-clean
-	rm -f $(allobjs) $(obj)/coreboot* .xcompile
+	rm -f $(allobjs) $(obj)/coreboot*
 	rm -f $(obj)/option_table.* $(obj)/crt0_includes.h $(obj)/ldscript
 	rm -f $(obj)/mainboard/$(MAINBOARDDIR)/static.c $(obj)/mainboard/$(MAINBOARDDIR)/config.py $(obj)/mainboard/$(MAINBOARDDIR)/static.dot
 	rm -f $(obj)/mainboard/$(MAINBOARDDIR)/auto.inc $(obj)/mainboard/$(MAINBOARDDIR)/crt0.s $(obj)/mainboard/$(MAINBOARDDIR)/crt0.disasm
@@ -297,7 +297,7 @@ clean: doxygen-clean
 	$(MAKE) -C util/sconfig clean
 
 distclean: clean
-	rm -rf $(obj)
+	rm -rf $(obj) .xcompile
 	rm -f .config .config.old ..config.tmp .kconfig.d .tmpconfig*
 
 update:
-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to