Re: [LTP] [PATCH v3] tst_test.sh: Normalize the locale

[Petr Vorel <pvorel@suse.cz>]

Hi all,


> network/tcp_cmds/tracepath/tracepath01.sh fails with LANG=ja_JP.UTF-8
> because it parses localized output. The below is an example of such
> output:
> $ tracepath localhost
>  1?: [LOCALHOST]                        0.040ミリ秒 pmtu 65536
>  1:  localhost                                             0.274ミリ秒 到達しました
>  1:  localhost                                             0.261ミリ秒 到達しました
>      要約: pmtu 65536 ホップ数 1 戻りホップ数 1

> It is necessary to normalize the locale to avoid such a problem.
> There are some tests do the normalization, but that is not
> comprehensive. Add code to normalize the locale to tst_test.sh so
> that it can cover more tests.

> The added code does the normalization by setting LC_ALL, which
> takes precedence to the other locale-related environment variables
> and does not require that "locale" command exists.

> Signed-off-by: Akihiko Odaki <akihiko.odaki@daynix.com>
> ---
>  testcases/lib/tst_test.sh | 2 ++
>  1 file changed, 2 insertions(+)

> diff --git a/testcases/lib/tst_test.sh b/testcases/lib/tst_test.sh
> index 28b7d12ba..5ebbe1d25 100644
> --- a/testcases/lib/tst_test.sh
> +++ b/testcases/lib/tst_test.sh
> @@ -831,3 +831,5 @@ if [ -z "$TST_NO_DEFAULT_RUN" ]; then
>  		fi
>  	fi
>  fi
> +
> +export LC_ALL=C

Thinking about it twice this might have impact on other tests.
Li, Cyril, any idea about it?

Other option would be to put this change just to tracepath01.sh.
And if we want to test just C locale (I'm really not sure, shouldn't
it be also in C API?

Kind regards,
Petr

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp