From 4dfeba3a0e53d4b697b07f9c10783c411e43ccdf Mon Sep 17 00:00:00 2001 From: Amaan Cheval Date: Fri, 9 Mar 2018 21:08:31 +0530 Subject: bootstrap: Use printf instead of echo -e for POSIX shells On POSIX compliant shells, echo does not have the -e option. This causes the "-e" to be echoed as well, causing potential buggy build processes. Example shell session: -> % sh $ echo -e "foo bar" -e foo bar $ According to POSIX, "\$" should be fine regardless due to the use of double-quotes[1]. However, since printf is recommended over echo anyway, we replace "echo -e" with printf where required. [1] http://pubs.opengroup.org/onlinepubs/009695399/utilities/xcu_chap02.html#tag_02_02_03 --- bootstrap | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/bootstrap b/bootstrap index eb6d4fc8f3..a0c6eda1f2 100755 --- a/bootstrap +++ b/bootstrap @@ -152,7 +152,7 @@ headers) if test x$d != x. ; then am_dir=`echo $dir | sed 's%[/-]%_%g'` am_dir="_$am_dir" - echo -e "\ninclude${am_dir}dir = \$(includedir)/$dir" >> "$tmp" + printf "\ninclude%sdir = \$(includedir)/$dir\n" "$am_dir" >> "$tmp" else am_dir="" echo "" >> "$tmp" @@ -161,7 +161,7 @@ headers) fi echo "include${am_dir}_HEADERS += $inc$j" >> "$tmp" if test $j = bsp.h ; then - echo -e "include_HEADERS += include/bspopts.h" >> "$tmp" + echo "include_HEADERS += include/bspopts.h" >> "$tmp" fi done done -- cgit v1.2.3