<div dir="ltr">Can you file a ticket for this? It seems like something which should be<div>considered for release branches.</div><div><br></div><div>What shell(s)/host(s) did this show up on?</div><div><br></div><div>Thanks.</div><div><br></div><div>--joel</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Mar 9, 2018 at 9:55 AM, Amaan Cheval <span dir="ltr"><<a href="mailto:amaan.cheval@gmail.com" target="_blank">amaan.cheval@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On POSIX compliant shells, echo does not have the -e option.<br>
This causes the "-e" to be echoed as well, causing potential buggy<br>
build processes.<br>
<br>
Example shell session:<br>
<br>
-> % sh<br>
$ echo -e "foo bar"<br>
-e foo bar<br>
$<br>
<br>
According to POSIX, "\$" should be fine regardless due to the use of<br>
double-quotes[1]. However, since printf is recommended over echo anyway,<br>
we replace "echo -e" with printf where required.<br>
<br>
[1] <a href="http://pubs.opengroup.org/onlinepubs/009695399/utilities/xcu_chap02.html#tag_02_02_03" rel="noreferrer" target="_blank">http://pubs.opengroup.org/<wbr>onlinepubs/009695399/<wbr>utilities/xcu_chap02.html#tag_<wbr>02_02_03</a><br>
---<br>
 bootstrap | 4 ++--<br>
 1 file changed, 2 insertions(+), 2 deletions(-)<br>
<br>
diff --git a/bootstrap b/bootstrap<br>
index eb6d4fc8f3..b94fa244ce 100755<br>
--- a/bootstrap<br>
+++ b/bootstrap<br>
@@ -152,7 +152,7 @@ headers)<br>
           if test x$d != x. ; then<br>
             am_dir=`echo $dir | sed 's%[/-]%_%g'`<br>
             am_dir="_$am_dir"<br>
-            echo -e "\ninclude${am_dir}dir = \$(includedir)/$dir" >> "$tmp"<br>
+            printf "\ninclude${am_dir}dir = \$(includedir)/$dir\n" >> "$tmp"<br>
           else<br>
             am_dir=""<br>
             echo "" >> "$tmp"<br>
@@ -161,7 +161,7 @@ headers)<br>
         fi<br>
         echo "include${am_dir}_HEADERS += $inc$j" >> "$tmp"<br>
         if test $j = bsp.h ; then<br>
-          echo -e "include_HEADERS += include/bspopts.h" >> "$tmp"<br>
+          echo "include_HEADERS += include/bspopts.h" >> "$tmp"<br>
         fi<br>
       done<br>
     done<br>
<span class="HOEnZb"><font color="#888888">--<br>
2.13.0<br>
<br>
______________________________<wbr>_________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/<wbr>mailman/listinfo/devel</a><br>
</font></span></blockquote></div><br></div>