[U-Boot] [PATCH] Introduce a new linker flag LDFLAGS_FINAL

[Mike Frysinger]

On Wednesday, February 16, 2011 13:58:07 Wolfgang Denk wrote:
> Haiying Wang wrote:
> > > We have some 524 PowerPC configurations in boards.cfg, and the compare
> > > must be done for _all_ off these, i. e. there are 2 x 524 pairs of
> > > files to be compared.  Did you really do this?
> > > 
> > > > it is ok to apply this patch, if you are ok with the name _FINAL?
> > 
> > Sorry, I did not get the 2x254 pairs of System.map and u-boot.map.
> > Running "MAKEALL powerpc" only left the last pair. Could you please show
> > me how I can keep the former 523 pairs?
> 
> I usually do this on the fly using shell scripting capabilities, i.e.
> something like this (copying some shell functions from MAKEALL):
> 
> -----------------------------------------------------------------------
> boards_by_field()
> {
>         awk \
>                 -v field="$1" \
>                 -v select="$2" \
>                 '($1 !~ /^#/ && $field == select) { print $1 }' \
>                 boards.cfg
> }
> boards_by_arch() { boards_by_field 2 "$@" ; }
> 
> for board in $(boards_by_arch powerpc) ; do
> ./MAKEALL $board
> cp System.map System.map.${board}.unpatched
> cp u-boot.map u-boot.map.${board}.unpatched
> done
> 
> ...apply patch...
> 
> for board in $(boards_by_arch powerpc) ; do
> ./MAKEALL $board
> cmp System.map.${board}.unpatched System.map || diff -u
> System.map.${board}.unpatched System.map >System.map.${board}.diff cmp
> u-boot.map.${board}.unpatched u-boot.map || diff -u
> u-boot.map.${board}.unpatched u-boot.map >u-boot.map.${board}.diff done

this is is generally useful, perhaps MAKEALL should grow a flag to back up the 
generated files with board-qualified names like the log files ...

then your script would be:
./MAKEALL <newflag> powerpc
<copy all generated files to old/>
<apply change>
./MAKEALL <newflag> powerpc
<compare results>
-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/20110217/82e6aae7/attachment.pgp