On Fri, Oct 19, 2012 at 03:19:19PM +0200, Marek Vasut wrote: > Dear Wolfgang Denk, > > [...] > > > Maybe we can sae one call to objdump by storing the intermediate > > result? > > I removed the other call in subsequent patch anyway, so this is fine now. > > > > + cd $(LNDIR) && $(LD) $(LDFLAGS) $(LDFLAGS_$(@F)) \ > > > + $$UNDEF_SYM $$UNDEF_LST $(__OBJS) \ > > > > Hm.... we use UNDEF_SYM and UNDEF_LST in exactly the same way? The > > why cannot we do this all in a single run, like: > > > > UNDEF_SYM=`$(OBJDUMP) -x $(LIBBOARD) $(LIBS) | \ > > sed -n -e 's/.*\($(SYM_PREFIX)__u_boot_cmd_.*\)/-u\1/p' \ > > -e 's/.*\($(SYM_PREFIX)_u_boot_list_.*\)/-u\1/p' | sort | uniq`; > \ > > cd $(LNDIR) && $(LD) $(LDFLAGS) $(LDFLAGS_$(@F)) $$UNDEF_SYM $(__OBJS) \ > > ... > > > > ? > > See above, otherwise I do agree.
It's not a big deal, in the end. But, why can't you dump the file once and pass two regexps to sed rather than dump it twice, regex it twice and reference two variables? -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
