[LTP] [PATCH 1/3] lib/tst_net.sh: Append 6 to the end of $TST_OPTS

[Petr Vorel]

Hi,

> On 05/07/2018 11:09 AM, Petr Vorel wrote:
> > Hi Xiao,

> >>> Good catch. Although I propose to get rid of ':' at the beginning for users of new shell
> >>> API (as it's IMHO better to see errors).
> >> Hi Petr,

> >> If getopts is in silent mode,  the invalid option is  placed  in OPTARG, so
> >> we can see errors by printing
> >> the value of OPTARG.  But it is reasonable for me to get rid of ':' at the
> >> beginning of optstring. :-)
> > I know, TBROK inform us, so it can stay how it is for legacy API.
> > I meant it for the new API, where is going to be removed (the TBROK as well) as error is
> > handled in tst_run in tst_test.sh.



> What about silence error in tst_test.sh? And remove preceding column in the
> tests (tcp_fastopen.sh, nfs_lib.sh). We handle the error already there, in
> tst_run (and in old API the scripts have TBROK), so there is no point
> printing one more redundant message that doesn't have LTP format.

> diff --git a/testcases/lib/tst_test.sh b/testcases/lib/tst_test.sh
> index 8d49d34..edac619 100644
> --- a/testcases/lib/tst_test.sh
> +++ b/testcases/lib/tst_test.sh
> @@ -265,11 +265,12 @@ tst_run()

>         OPTIND=1

> -       while getopts "hi:$TST_OPTS" name $TST_ARGS; do
> +       while getopts ":hi:$TST_OPTS" name $TST_ARGS; do
>                 case $name in
>                 'h') tst_usage; exit 0;;
>                 'i') TST_ITERATIONS=$OPTARG;;
> -               '?') tst_usage; exit 2;;
> +               '?') tst_usage
> +                    tst_brk TBROK "invalid option: '$OPTARG'";;
>                 *) $TST_PARSE_ARGS "$name" "$OPTARG";;
>                 esac
+1 as it makes sense to unify things.
It's just a bit faster to have printed the name of the script (if we don't use ':').

BTW this change affect only tests using new API (i.e. not tcp_fastopen_run.sh, nor nfs_lib.sh which still use legacy API).
I'm working on rewriting tests into new API.


Kind regards,
Petr