From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Sat, 20 Oct 2012 10:46:22 -0700 Subject: [U-Boot] [PATCH 1/6 V4] common: Add symbol handling for generic lists into Makefile In-Reply-To: <201210191519.19153.marex@denx.de> References: <1350073626-17024-2-git-send-email-marex@denx.de> <1350470636-7625-1-git-send-email-marex@denx.de> <20121018194021.7932B204030@gemini.denx.de> <201210191519.19153.marex@denx.de> Message-ID: <20121020174622.GH5854@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: