From: Wolfgang Denk Date: Fri, 07 Dec 2018 16:21:27 +0100 Subject: [U-Boot] [PATCH] Enable expression support for CONFIG_BOARD_SIZE_LIMIT In-Reply-To: References: <20181204154035.16299-1-wd@denx.de> <20181205095202.027A5242268@gemini.denx.de> <20181206142315.53AF6242268@gemini.denx.de> Message-ID: <20181207152127.923192400DF@gemini.denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Dear Fabio, In message you wrote: > > > /bin/sh: 1: arithmetic expression: expecting primary: ""(1 * 1024)"" D*mn. I really thought I had tried this in a dash based environment, too. Sorry for causing such confusion. > SHELL = /bin/bash" Yes, if this is really a bash only feature that would be an easy way to fix it. > And by forcing the SHELL variable to bash, then your patch works fine here: Yes, this would work - but I'm not sure if everybody would appreciate such a change? This should also work - replace the line @(echo $$(($(CONFIG_BOARD_SIZE_LIMIT))); wc -c $@ ) | \ by @(awk "END { print $$(echo $(CONFIG_BOARD_SIZE_LIMIT)) }" /dev/null; wc -c $@ ) | \ Can you please try this out? Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de Everyting looks interesting until you do it. Then you find it's just another job. - Terry Pratchett, _Moving Pictures_