[U-Boot] [PATCH] makefiles: fixes/cleanup for building build tools

Mike Frysinger vapier at gentoo.org
Tue Oct 27 00:57:50 CET 2009


On Monday 26 October 2009 18:17:03 Scott Wood wrote:
> Currently, some of the tools instead set CC to be HOSTCC in order to re-use
> some pattern rules -- but this fails when the user overrides CC on the make
> command line.  Also, the HOSTCFLAGS in tools/Makefile are currently not
> being used because config.mk overwrites them.
> 
> This patch adds static pattern rules for files that have been requested to
> be built with the native compiler using $(HOSTSRCS) and $(HOSTOBJS), and
> converts the tools to use them.

your new easylogo rule lacks HOSTLDFLAGS ...

perhaps it would make more sense to create a HOSTCOMPILE/HOSTLINK (or 
whatever) variable so this kind of thing isnt missed ?
HOSTCOMPILE = $(HOSTCC) $(HOSTCFLAGS)
HOSTLINK = $(HOSTCOMPILE) $(HOSTLDFLAGS)

also, PEDCFLAGS seems at odds with the rest of your consistency changes.  how 
about naming it HOSTCFLAGS+PED ?  then you would have a new var:
HOSTCOMPILE+PED = $(HOSTCC) $(HOSTCFLAGS+PED)

> It restores easylogo to using the host compiler, which was broken by commit
> 38d299c2db81bd889c601b5dfc12c4e83ef83333 (if this was an intentional
>  change, please let me know -- but it seems to be a build tool).

it was intentional, but for different reasons.  easylogo isnt integrated into 
the u-boot build system, so in order to compile things in there, you had to go 
into the subdir and manually run `make`.  if it were integrated into the build 
system like all other tools, then converting to host tools is fine.  but 
unless i missed something, it doesnt appear to be ?  and now, going into the 
subdir and running `make` wont work either ...
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20091026/c7ec0167/attachment.pgp 


More information about the U-Boot mailing list