build: image: use printf for ModelNameLimit16
authorHuangbin Zhan <zhanhb88@gmail.com>
Tue, 29 Mar 2022 16:15:29 +0000 (00:15 +0800)
committerChristian Marangi <ansuelsmth@gmail.com>
Thu, 13 Oct 2022 13:57:35 +0000 (15:57 +0200)
commit2f8ac8bcea879542b2bc414a108c3aff166d5ef5
tree09169e1c52519cd77e314ed2cea7affa3ea07cac
parent88803cb0e6e42ea00bd7c3dc8838e660239d3f16
build: image: use printf for ModelNameLimit16

Fix syntax error on macos, for substr is undefined results according to
the POSIX standard.

From expr on macos:
 According to the POSIX standard, the use of string arguments length,
 substr, index, or match produces undefined results.  In this version of
 expr, these arguments are treated just as their respective string values.

By a simple test Makefile:

define ModelNameLimit16
$(shell expr substr "$(word 2, $(subst _, ,$(1)))" 1 16)
endef

define ModelNameLimit16_2
$(shell printf %.16s "$(word 2, $(subst _, ,$(1)))")
endef

hello:
echo $(call ModelNameLimit16, technicolor_tg582n-telecom-italia)
echo $(call ModelNameLimit16_2, technicolor_tg582n-telecom-italia)

The same output is produced.

echo tg582n-telecom-i
tg582n-telecom-i
echo tg582n-telecom-i
tg582n-telecom-i

Signed-off-by: Huangbin Zhan <zhanhb88@gmail.com>
[ wrap commit description to 80 columns and improve it ]
Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
include/image-commands.mk