[PATCH] bootstrap: Use printf instead of echo -e for POSIX shells

Joel Sherrill joel at rtems.org
Fri Mar 9 16:17:16 UTC 2018


Can you file a ticket for this? It seems like something which should be
considered for release branches.

What shell(s)/host(s) did this show up on?

Thanks.

--joel

On Fri, Mar 9, 2018 at 9:55 AM, Amaan Cheval <amaan.cheval at gmail.com> wrote:

> 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..b94fa244ce 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${am_dir}dir = \$(includedir)/$dir\n" >>
> "$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
> --
> 2.13.0
>
> _______________________________________________
> devel mailing list
> devel at rtems.org
> http://lists.rtems.org/mailman/listinfo/devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/devel/attachments/20180309/92d7bcd2/attachment-0002.html>


More information about the devel mailing list