From: Jo-Philipp Wich <jow@openwrt.org>
Date: Thu, 13 Feb 2014 21:19:00 +0000 (+0000)
Subject: ar71xx: fix mtdpartsize macro in image Makefile (#14961)
X-Git-Tag: reboot~7947
X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=b1d7ebae09622252906f7a7da1af2e3f30f6cf0e;p=openwrt%2Fstaging%2Fblogic.git

ar71xx: fix mtdpartsize macro in image Makefile (#14961)

The mtdpartsize macro triggers shell errors on various Linux distributions
when the partname argument $(1) does not appear within the partmap $(2).

Change the sed pattern to only emit anything if a successful substitution
occured and only evaluate the arithmetic expression if something was printed
by the sed program.

Signed-off-by: Jo-Philipp Wich <jow@openwrt.org>

SVN-Revision: 39583
---

diff --git a/target/linux/ar71xx/image/Makefile b/target/linux/ar71xx/image/Makefile
index 1f181f4ce2aa..9076f22ff087 100644
--- a/target/linux/ar71xx/image/Makefile
+++ b/target/linux/ar71xx/image/Makefile
@@ -36,7 +36,7 @@ $(if $(1),board=$(1) )$(if $(2),console=$(2)$(COMMA)$(3))
 endef
 
 define mtdpartsize
-$(shell echo $$((`echo '$(2)' | sed -e 's/.*[:$(COMMA)]\([0-9]*\)k[@]*[0-9a-zx]*($(1)).*/\1/'` * 1024)))
+$(shell sz=`echo '$(2)' | sed -ne 's/.*[:$(COMMA)]\([0-9]*\)k[@]*[0-9a-zx]*($(1)).*/\1/p'`; [ -n "$$sz" ] && echo $$(($$sz * 1024)))
 endef
 
 SINGLE_PROFILES:=