<div dir="ltr">I like the sound of next time. :)</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Mar 9, 2018 at 3:37 PM, 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">Cheers!<br>
<br>
I'll keep that in mind next time. :)<br>
<div class="HOEnZb"><div class="h5"><br>
On Sat, Mar 10, 2018 at 12:57 AM Joel Sherrill <<a href="mailto:joel@rtems.org">joel@rtems.org</a>> wrote:<br>
<br>
> Awesome! Good catch! I have pushed the patch and closed the ticket.<br>
<br>
> I should have spotted it but if you make the last line of your git message<br>
> something like:<br>
<br>
> Closes #NNN.<br>
> Updates #NNN.<br>
<br>
> where NNN is the ticket number, the git hook for Trac will automatically<br>
> update the ticket.<br>
<br>
> On Fri, Mar 9, 2018 at 11:00 AM, Amaan Cheval <<a href="mailto:amaan.cheval@gmail.com">amaan.cheval@gmail.com</a>><br>
wrote:<br>
<br>
>> On Fri, Mar 9, 2018 at 9:47 PM Joel Sherrill <<a href="mailto:joel@rtems.org">joel@rtems.org</a>> wrote:<br>
<br>
>> > Can you file a ticket for this? It seems like something which should be<br>
>> > considered for release branches.<br>
<br>
>> Sure. Here it is:<br>
>> <a href="https://devel.rtems.org/ticket/3328" rel="noreferrer" target="_blank">https://devel.rtems.org/<wbr>ticket/3328</a><br>
<br>
>> Note, however, that the use of "echo -e" was introduced in 2afb22b,<br>
>> to remove the make preinstall step[1], so it isn't in any of the current<br>
>> releases as far as I can tell, only on master.<br>
<br>
>> [1] <a href="https://devel.rtems.org/ticket/3254#comment:78" rel="noreferrer" target="_blank">https://devel.rtems.org/<wbr>ticket/3254#comment:78</a><br>
<br>
<br>
>> > What shell(s)/host(s) did this show up on?<br>
<br>
>> I use zsh on Ubuntu 16.04 LTS (4.4.0-78-generic x86_64 GNU/Linux),<br>
>> but since #!/bin/sh is specified in bootstrap, my use of zsh (which does<br>
>> support -e) is negated by my built-in sh.<br>
<br>
>> The "-e" option may or may not be present even on sh or bash, apparently:<br>
<br>
<a href="https://unix.stackexchange.com/questions/88307/escape-sequences-with-echo-e-in-different-shells#comment131955_88310" rel="noreferrer" target="_blank">https://unix.stackexchange.<wbr>com/questions/88307/escape-<wbr>sequences-with-echo-e-in-<wbr>different-shells#<wbr>comment131955_88310</a><br>
<br>
<br>
>> > Thanks.<br>
<br>
>> > --joel<br>
<br>
>> > On Fri, Mar 9, 2018 at 9:55 AM, Amaan Cheval <<a href="mailto:amaan.cheval@gmail.com">amaan.cheval@gmail.com</a>><br>
>> wrote:<br>
<br>
>> >> 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<br>
anyway,<br>
>> >> we replace "echo -e" with printf where required.<br>
<br>
>> >> [1]<br>
<br>
<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" >><br>
>> "$tmp"<br>
>> >> +            printf "\ninclude${am_dir}dir = \$(includedir)/$dir\n" >><br>
>> "$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>
>> >> --<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>
</div></div></blockquote></div><br></div>