diff options
author | Amaan Cheval <amaan.cheval@gmail.com> | 2018-03-09 21:08:31 +0530 |
---|---|---|
committer | Joel Sherrill <joel@rtems.org> | 2018-03-09 13:16:01 -0600 |
commit | 4dfeba3a0e53d4b697b07f9c10783c411e43ccdf (patch) | |
tree | 47fd01badb8648ea78ce17f4fc1a656d894296c3 /bootstrap | |
parent | 16f4661fd1fd13b7b5ce63756e0cbfd20843aa75 (diff) |
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
Diffstat (limited to 'bootstrap')
-rwxr-xr-x | bootstrap | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -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 |