[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/2] iotests: Set LC_ALL=C for sort
From: |
Alex Bennée |
Subject: |
Re: [PATCH 2/2] iotests: Set LC_ALL=C for sort |
Date: |
Sat, 11 Jul 2020 09:57:55 +0100 |
User-agent: |
mu4e 1.5.4; emacs 28.0.50 |
Max Reitz <mreitz@redhat.com> writes:
> Otherwise the result is basically unpredictable.
>
> (Note that the precise environment variable to control sorting order is
> LC_COLLATE, but LC_ALL overrides LC_COLLATE, and we do not want the
> sorting order to be messed up if LC_ALL is set in the environment.)
>
> Reported-by: John Snow <jsnow@redhat.com>
> Signed-off-by: Max Reitz <mreitz@redhat.com>
Queued to pr/100720-testing-and-misc-2, thanks.
I've skipped patch 1/2 for now as I have an alternative fix but we can
switch it back if you prefer?
> ---
> tests/qemu-iotests/common.filter | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/qemu-iotests/common.filter
> b/tests/qemu-iotests/common.filter
> index 345c3ca03e..4fd5c29b2a 100644
> --- a/tests/qemu-iotests/common.filter
> +++ b/tests/qemu-iotests/common.filter
> @@ -177,7 +177,7 @@ _do_filter_img_create()
> -e 's/^\(data_file\)/3-\1/' \
> -e 's/^\(encryption\)/4-\1/' \
> -e 's/^\(preallocation\)/8-\1/' \
> - | sort \
> + | LC_ALL=C sort \
> | $SED -e 's/^[0-9]-//' \
> | tr '\n\0' ' \n' \
> | $SED -e 's/^ *$//' -e 's/ *$//'
--
Alex Bennée