[LTP] [PATCH ltp] runltp: fix shellcheck error for sending email
Petr Vorel
pvorel@suse.cz
Mon Jul 23 13:31:58 CEST 2018
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]
> Also change '-o' to '&&' according to the comment and context.
> Signed-off-by: Yixin Zhang <yixin.zhang@intel.com>
> ---
> runltp | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
> diff --git a/runltp b/runltp
> index 51357fadf..c6e5b03e8 100755
> --- a/runltp
> +++ b/runltp
> @@ -891,7 +891,7 @@ main()
> fi
> if [ "$ALT_EMAIL_OUT" -eq 1 ] ; then ## User wants reports to be e-mailed
> - if [ [ ! "$HTMLFILE_NAME" ] -o [ ! "$OUTPUTFILE_NAME" ] -o [ ! "$LOGFILE_NAME" ] ] ; then
> + if [ -z "$HTMLFILE_NAME" ] && [ -z "$OUTPUTFILE_NAME" ] && [ -z "$LOGFILE_NAME" ]; then
Actually this if clause shouldn't be here as $LOGFILE_NAME is always defined:
l)
LOGFILE_NAME="$OPTARG"
case $OPTARG in
/*)
LOGFILE="-l $OPTARG" ;;
*)
LOGFILE="-l $LTPROOT/results/$OPTARG"
ALT_DIR_RES=1 ;;
esac ;;
....
if [ ! "$LOGFILE" ]; then ## User has not mentioned about Log File name
LOGFILE_NAME="$DEFAULT_FILE_NAME_GENERATION_TIME"
Could you please just delete it?
I'm looking forward the day when this crazy script is replaced by new runner.
Kind regards,
Petr
More information about the ltp
mailing list