[LTP] [PATCH v2 ltp] runltp: fix shellcheck error for sending email

[Petr Vorel]

Hi Yixin,

> runltp:894:11: error: Couldn't parse this test expression. [SC1073]
> runltp:894:13: error: If grouping expressions inside [..], use \( ..\). [SC1026]
> runltp:894:15: error: Expected test to end here (don't wrap commands in []/[[]]).
>                       Fix any mentioned problems and try again. [SC1072]

> Change '-o' to '&&' according to the comment and context.
> Then as "$LOGFILE_NAME" is always not empty, remove below "if" statement
> and update indentation accordingly:

> if [ -z "$HTMLFILE_NAME" ] && [ -z "$OUTPUTFILE_NAME" ] && [ -z "$LOGFILE_NAME" ]; then
>   ##User does not have output/logs/html-output, nothing to be mailed in this situation
>   echo "Nothing to be mailed here...."

Just to warn you this file si deprecated as Cyril Hrubis works on new runner [1] [2]
So it might be better test it than invest time into something which is going to
be deleted.

> Signed-off-by: Yixin Zhang <yixin.zhang@intel.com>
> ---
>  runltp | 139 +++++++++++++++++++++++++++++++----------------------------------
>  1 file changed, 67 insertions(+), 72 deletions(-)

...
> +    if [ "$ALT_EMAIL_OUT" -eq 1 ] ; then                    ## User wants reports to be e-mailed
> +        TAR_FILE_NAME=LTP_RUN_$version_date$DEFAULT_FILE_NAME_GENERATION_TIME.tar
> +        if [ "$HTMLFILE_NAME" ] ; then                      ## HTML file Exists
> +            if [ "$ALT_HTML_OUT" -ne 1 ] ; then             ## The HTML file path is absolute and not $LTPROOT/output
> +                mkdir -p $LTPROOT/output                    ## We need to create this Directory
> +                cp $HTMLFILE_NAME $LTPROOT/output/
> +            fi
> +        fi
> +        if [ "$OUTPUTFILE_NAME" ] ; then                    ## Output file exists
> +            if [ "$ALT_DIR_OUT" -ne 1 ] ; then              ## The Output file path is absolute and not $LTPROOT/output
> +                mkdir -p $LTPROOT/output                    ## We need to create this Directory
> +                cp $OUTPUTFILE_NAME $LTPROOT/output/
> +            fi
> +        fi
> +        if [ "$LOGFILE_NAME" ] ; then                       ## Log file exists
As I wrote in comment to v1 [3], $LOGFILE_NAME is always defined, so no need to check for it.
And the same is for for $OUTPUTFILE_NAME

> +            if [ "$ALT_DIR_RES" -ne 1 ] ; then              ## The Log file path is absolute and not $LTPROOT/results
> +                mkdir -p $LTPROOT/results                   ## We need to create this Directory
> +                cp $LOGFILE_NAME $LTPROOT/results/
> +            fi
> +        fi


Kind regards,
Petr

[1] https://hackweek.suse.com/17/projects/prototype-new-ltp-upstream-runltp-script
[2] https://github.com/metan-ucw/ltp
[3] http://lists.linux.it/pipermail/ltp/2018-July/008783.html