Re: [LTP] [PATCH 2/2] runltp: remove bashisms

[Sedat D. <sed...@gm...>]

On Thu, Aug 29, 2013 at 9:27 AM, Anders Roxell <and...@li...> wrote:
> Signed-off-by: Anders Roxell <and...@li...>
> ---
>  runltp | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/runltp b/runltp
> index a419bf8..970c4ee 100755
> --- a/runltp
> +++ b/runltp
> @@ -733,15 +733,15 @@ main()
>      #$PAN_COMMAND #Duplicated code here, because otherwise if we fail, only "PAN_COMMAND" gets output
>
>      ## Display the Output/Log/Failed/HTML file names here
> -    echo -e "LOG File: \c"
> +    echo -n "LOG File: "
>      echo $LOGFILE | cut -b4-
>
>      if [ "$OUTPUTFILE" ]; then
> -       echo -e "OUTPUT File: \c"
> +       echo -n "OUTPUT File: "
>         echo $OUTPUTFILE | cut -b4-
>      fi
>
> -    echo -e "FAILED COMMAND File: \c"
> +    echo -n "FAILED COMMAND File: "
>      echo $FAILCMDFILE | cut -b4-
>
>      if [ "$HTMLFILE" ]; then
> --
> 1.8.1.2
>

Yesterday, I looked through diverse GIT repos where I saw a commit
saying that "echo" (dunno if it was "-e" or "-n") is not
POSIX-compliant and "printf" should be used.
I am no POSIX guy, but maybe someone can confirm this?

- Sedat -

P.S.: Google gave me...

[1] http://pubs.opengroup.org/onlinepubs/009696799/utilities/echo.html
[2] https://bugs.kde.org/show_bug.cgi?id=160368