diff options
author | Pádraig Brady <P@draigBrady.com> | 2010-02-26 15:33:16 +0000 |
---|---|---|
committer | Pádraig Brady <P@draigBrady.com> | 2010-03-01 10:58:02 +0000 |
commit | 59e2e55d0f154a388adc9bac37d2b45f2ba971f8 (patch) | |
tree | 28b45daf114e30b79db3f9110683ef6d6b48c5aa /tests | |
parent | 2590389aabc3ac2e88c56a13d28fa3caa22a5b46 (diff) | |
download | coreutils-59e2e55d0f154a388adc9bac37d2b45f2ba971f8.tar.xz |
sort: fix issues with month sorting in some locales
* src/sort.c (char fold_toupper[]): Change to unsigned
so as the correct comparisons are made in getmonth().
This fixes unibyte locales where abbreviated months
have characters that are > 0x7F, but it also works for
multibyte locales with the caveat that multibyte characters
are matched case sensitively.
With this change, the following example sorts correctly:
$ echo -e "1 márta\n2 Feabhra" | LANG=ga_IE.utf8 sort -k2,2M
2 Feabhra
1 márta
* src/sort.c (inittables): Since we ignore blanks around months
in the input, don't include them when they're present in the locale.
With this change, the following example sorts correctly:
$ echo -e "1 2月\n2 1月" | LANG=ja_JP.utf8 sort -k2,2M
2 1月
1 2月
* tests/misc/sort-month: A new test to exercise the above cases.
* tests/Makefile.am: Reference the new test.
* NEWS: Mention the fix.
Diffstat (limited to 'tests')
-rw-r--r-- | tests/Makefile.am | 1 | ||||
-rwxr-xr-x | tests/misc/sort-month | 38 |
2 files changed, 39 insertions, 0 deletions
diff --git a/tests/Makefile.am b/tests/Makefile.am index 68b3f71b7..db1610d6b 100644 --- a/tests/Makefile.am +++ b/tests/Makefile.am @@ -225,6 +225,7 @@ TESTS = \ misc/sort-files0-from \ misc/sort-merge \ misc/sort-merge-fdlimit \ + misc/sort-month \ misc/sort-rand \ misc/sort-version \ misc/split-a \ diff --git a/tests/misc/sort-month b/tests/misc/sort-month new file mode 100755 index 000000000..aee5215d5 --- /dev/null +++ b/tests/misc/sort-month @@ -0,0 +1,38 @@ +#!/bin/sh +# Test sorting of abbreviated months from the locale + +# Copyright (C) 2010 Free Software Foundation, Inc. + +# This program is free software: you can redistribute it and/or modify +# it under the terms of the GNU General Public License as published by +# the Free Software Foundation, either version 3 of the License, or +# (at your option) any later version. + +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. + +# You should have received a copy of the GNU General Public License +# along with this program. If not, see <http://www.gnu.org/licenses/>. + +if test "$VERBOSE" = yes; then + set -x + sort --version +fi + +. $srcdir/test-lib.sh + +locale --version >/dev/null 2>&1 || + skip_test_ 'The locale utility is not present' + +# C will be used if the locale is not present +for LOC in "$LOCALE_FR" "$LOCALE_FR_UTF8" "ja_JP.utf8"; do + mon="$(LC_ALL="$LOC" locale abmon 2>/dev/null);" + smon=$(LC_ALL="$LOC" locale abmon 2>/dev/null | + tr ';' '\n' | shuf | nl | LC_ALL="$LOC" sort -b -k2,2M | + cut -f2 | tr '\n' ';') + test "$mon" = "$smon" || { fail=1; break; } +done + +Exit $fail |