dsb_DE locale: Fix syntax error and add tests (bug 23208).
Commit Message
Fixed syntax error in the collation rules of Lower Sorbian language.
Collation test added in order to test the bugs like this early.
Reported-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>
[BZ #23208]
* localedata/Makefile (test-input): Add dsb_DE.UTF-8.
(LOCALES): Likewise.
* localedata/dsb_DE.UTF-8.in: New file.
* localedata/locales/dsb_DE (LC_COLLATE): Fix syntax error.
---
localedata/Makefile | 2 ++
localedata/dsb_DE.UTF-8.in | 73 ++++++++++++++++++++++++++++++++++++++++++++++
localedata/locales/dsb_DE | 4 +--
3 files changed, 77 insertions(+), 2 deletions(-)
create mode 100644 localedata/dsb_DE.UTF-8.in
Comments
Rafal Luzynski <digitalfreak@lingonborough.com> writes:
> Fixed syntax error in the collation rules of Lower Sorbian language.
> Collation test added in order to test the bugs like this early.
>
> Reported-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>
>
> [BZ #23208]
> * localedata/Makefile (test-input): Add dsb_DE.UTF-8.
> (LOCALES): Likewise.
> * localedata/dsb_DE.UTF-8.in: New file.
> * localedata/locales/dsb_DE (LC_COLLATE): Fix syntax error.
I confirmed this patch does fix the issue.
Thanks!
13.07.2018 17:02 Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com> wrote:
>
> Rafal Luzynski <digitalfreak@lingonborough.com> writes:
>
> > Fixed syntax error in the collation rules of Lower Sorbian language.
> > Collation test added in order to test the bugs like this early.
> >
> > Reported-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>
> >
> > [BZ #23208]
> > * localedata/Makefile (test-input): Add dsb_DE.UTF-8.
> > (LOCALES): Likewise.
> > * localedata/dsb_DE.UTF-8.in: New file.
> > * localedata/locales/dsb_DE (LC_COLLATE): Fix syntax error.
>
> I confirmed this patch does fix the issue.
>
> Thanks!
Since the patch fixes an actual bug and changes of the locale data are
usually not harmful for the rest of the project I assume this review
is sufficient and I'm going to push this patch now.
Regards,
Rafal
11.07.2018 23:59 Rafal Luzynski <digitalfreak@lingonborough.com> wrote:
>
> Fixed syntax error in the collation rules of Lower Sorbian language.
> Collation test added in order to test the bugs like this early.
>
> Reported-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>
>
> [BZ #23208]
> * localedata/Makefile (test-input): Add dsb_DE.UTF-8.
> (LOCALES): Likewise.
> * localedata/dsb_DE.UTF-8.in: New file.
> * localedata/locales/dsb_DE (LC_COLLATE): Fix syntax error.
> ---
This patch has been pushed to master.
Regards,
Rafal
@@ -55,6 +55,7 @@ test-input := \
cy_GB.UTF-8 \
da_DK.ISO-8859-1 \
de_DE.ISO-8859-1 \
+ dsb_DE.UTF-8 \
dz_BT.UTF-8 \
en_US.ISO-8859-1 \
eo.UTF-8 \
@@ -200,6 +201,7 @@ LOCALES := \
da_DK.ISO-8859-1 \
de_DE.ISO-8859-1 \
de_DE.UTF-8 \
+ dsb_DE.UTF-8 \
dz_BT.UTF-8 \
en_GB.UTF-8 \
en_US.ANSI_X3.4-1968 \
new file mode 100644
@@ -0,0 +1,73 @@
+a
+A
+b
+B
+c
+C
+Ä
+Č
+ć
+d
+D
+dź
+Dź
+e
+E
+Ä›
+f
+F
+g
+G
+h
+H
+ch
+Ch
+i
+I
+j
+J
+k
+K
+Å‚
+Å
+l
+L
+m
+M
+n
+N
+Å„
+o
+O
+ó
+p
+P
+q
+Q
+r
+R
+Å•
+s
+S
+Å¡
+Å
+Å›
+Åš
+t
+T
+u
+U
+v
+V
+w
+W
+x
+X
+y
+Y
+z
+Z
+ž
+Ž
+ź
+Ź
@@ -77,8 +77,8 @@ collating-symbol <d-z-acute-digraph>
collating-symbol <e-caron>
collating-symbol <c-h-digraph>
collating-symbol <l-stroke>
-collating symbol <n-acute>
-collating symbol <o-acute>
+collating-symbol <n-acute>
+collating-symbol <o-acute>
collating-symbol <r-acute>
collating-symbol <s-caron>
collating-symbol <s-acute>